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.

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

Allow XMPP Server to be different from JID domain #41

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

stevenlivz
Copy link

Added support for specifying the XMPP server to connect to when it s different from the XMPP hostname in the JID.

… different from the XMPP hostname in the Jid.
@marchanddesable
Copy link

Hi,

You have updated the nuget package too?
Because I don't think this update is in the current version of Sharp.Xmpp hosted in the nuget server.

Thanks.

stevenlivz and others added 19 commits May 6, 2017 17:54
* Sharp.Xmpp.csproj:
* Sharp.Xmpp.userprefs:
Make it possible to retrieve the room which spawned GroupPresenceEventArgs
@ddrinka
Copy link

ddrinka commented Apr 5, 2019

@stevenlivz This is a really useful core PR, but it looks like your entire development process get pulled into it.

Usually the best approach to Git Flow is to create a branch in your repo for a PR, do the changes, and then create the PR from that branch. It looks like you created a PR from your master branch, so as you made additional changes, the PR grew out of control.

Would you mind cherry picking the first commit (9d5477a) off into a new branch and creating a new PR? Hopefully that could resolve folks problems (#65, #58, #34), and mine as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants