Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Xmpp async #6640

Merged
merged 4 commits into from
Oct 13, 2018
Merged

Xmpp async #6640

merged 4 commits into from
Oct 13, 2018

Conversation

flowolf
Copy link
Contributor

@flowolf flowolf commented Oct 9, 2018 •
edited
Loading

Description:

xmpp async support with new lib.
also:

  • TLSv1.1
  • TLSv1.2
  • resource option for JID

Pull request in home-assistant (if applicable): home-assistant/core#17283

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follow the standards.

@ghost ghost added the to-do label Oct 9, 2018
@flowolf flowolf mentioned this pull request Oct 9, 2018
Merged
8 tasks
@frenck frenck added Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! new-feature This PR adds documentation for a new Home Assistant feature to an existing integration ready-for-review This PR needs to be reviewed next This PR goes into the next branch has-parent This PR has a parent PR in a other repo in-progress This PR/Issue is currently being worked on and removed to-do ready-for-review This PR needs to be reviewed labels Oct 9, 2018
@flowolf flowolf changed the title [WIP] Xmpp async Xmpp async Oct 11, 2018
fabaff
fabaff previously approved these changes Oct 13, 2018
View reviewed changes
Copy link
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

Thanks 🐦

@flowolf
Copy link
Contributor Author

flowolf commented Oct 13, 2018

@fabaff this will be in 0.81, won't it? so the version should be 0.81.

@fabaff
Copy link
Member

fabaff commented Oct 13, 2018

Indeed, thanks.

@fabaff
Copy link
Member

fabaff commented Oct 13, 2018

I will fix it after merging while updating the configuration section.

@fabaff fabaff merged commit 0573c83 into home-assistant:next Oct 13, 2018
@ghost ghost removed the in-progress This PR/Issue is currently being worked on label Oct 13, 2018
@fabaff fabaff mentioned this pull request Oct 13, 2018
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabaff fabaff fabaff approved these changes

Assignees

@flowolf flowolf

Labels
Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! has-parent This PR has a parent PR in a other repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@flowolf @fabaff @frenck