-
-
Notifications
You must be signed in to change notification settings - Fork 585
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support global account_data #150
Conversation
/** | ||
* Set account data event for the current user. | ||
* @param {string} eventType The event type | ||
* @param {Object} the contents object for the event |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For JSDoc to pick it up sanely, the thing after the type has to be the variable name, hence this should be:
* @param {Object} contents the object for the event
Overall, beautiful :) just needs some clarification on docs. |
@kegsay: PTAL |
LGTM |
@@ -4001,6 +4015,17 @@ module.exports.CRYPTO_ENABLED = CRYPTO_ENABLED; | |||
* @param {module:client~DeviceInfo} device information about the verified device | |||
*/ | |||
|
|||
/** | |||
* Fires whenever new user-scoped account_data is added. | |||
* @event module:client~MatrixClient#"Room" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be module:client~MatrixClient#"accountData"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bah didn't spot that! Yes it should be.
No description provided.