-
-
Notifications
You must be signed in to change notification settings - Fork 119
improve jsonrpc js docs #6168
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
improve jsonrpc js docs #6168
Conversation
…ng more useful displayed on npm and on https://js.jsonrpc.delta.chat
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.
this will be shown as start page for https://js.jsonrpc.delta.chat
|
needs rebase |
| } | ||
|
|
||
| /// Returns a vector of contact IDs for given chat ID where the contact is not SELF. | ||
| pub async fn get_other_chat_contacts(context: &Context, chat_id: ChatId) -> Result<Vec<ContactId>> { |
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.
Looks like this landed in the PR accidentally?
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.
It is not the only thing that doesn't belong into it. I have to check everything again, and maybe it's easiest to copy wanted changes over to a fresh pr instead of dealing with git and rebasing.
DeltaChattoWebsocketDeltaChatEventandContextEventstype to include them in the documentation.@deltachat/jsonrpc-client. so there is something more useful displayed on npm and on https://js.jsonrpc.delta.chat