Skip to content

Remove references to simple-xmpp #357

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

Merged
merged 10 commits into from
Apr 15, 2021
Merged

Remove references to simple-xmpp #357

merged 10 commits into from
Apr 15, 2021

Conversation

silverbucket
Copy link
Member

@silverbucket silverbucket commented Apr 13, 2021

Updates API.md for new function signatures and removes references to simple-xmpp

Copy link
Contributor

@galfert galfert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a couple comments.

silverbucket and others added 7 commits April 15, 2021 00:04
Co-authored-by: Garret Alfert <alfert@wevelop.de>
Co-authored-by: Garret Alfert <alfert@wevelop.de>
Co-authored-by: Garret Alfert <alfert@wevelop.de>
Co-authored-by: Garret Alfert <alfert@wevelop.de>
Co-authored-by: Garret Alfert <alfert@wevelop.de>
Co-authored-by: Garret Alfert <alfert@wevelop.de>
Co-authored-by: Garret Alfert <alfert@wevelop.de>
@silverbucket
Copy link
Member Author

@galfert the API doc generation capitalizes that xmpP for some reason, it's annoying. Anyway, I've updated based on your comments. Should be ready for review again after the tests pass.

Copy link
Contributor

@galfert galfert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@raucao raucao changed the title remove references to simple-xmpp Remove references to simple-xmpp Apr 15, 2021
@silverbucket silverbucket merged commit 4a8b704 into master Apr 15, 2021
@galfert galfert deleted the cleanup-simple-xmpp branch April 15, 2021 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants