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

Changed XMPP configuration variable #1438

Merged
merged 3 commits into from
Mar 10, 2017
Merged

Changed XMPP configuration variable #1438

merged 3 commits into from
Mar 10, 2017

Conversation

MattGill98
Copy link
Contributor

Changed XMPP configuration variable named roomName to roomId and text from Name to ID.

@MattGill98
Copy link
Contributor Author

jenkins test please

@payara-ci
Copy link
Contributor

All tests have passed

@mulderbaba
Copy link
Contributor

@MattGill98 so this makes PR#1410 obsolete?

@mulderbaba mulderbaba added this to the Payara 172 milestone Mar 6, 2017
@MattGill98
Copy link
Contributor Author

It does, sorry I didn't check that PR before making mine! I think this PR covers all changes from #1410 though.

@MattGill98
Copy link
Contributor Author

Jenkins test please

@payara-ci
Copy link
Contributor

All tests have passed

@mulderbaba
Copy link
Contributor

mulderbaba commented Mar 7, 2017

cool @MattGill98 I'm closing the other one, PR#1410, then.

@smillidge smillidge merged commit a229aa4 into payara:master Mar 10, 2017
Pandrex247 pushed a commit that referenced this pull request Mar 27, 2017
* Changed XMPP configuration variable named roomName to roomId and text from Name to ID

* Updated variable name in Java files as well

* Changed 'a XMPP notifier' to 'an XMPP notifier'
Pandrex247 pushed a commit that referenced this pull request Mar 27, 2017
* Changed XMPP configuration variable named roomName to roomId and text from Name to ID

* Updated variable name in Java files as well

* Changed 'a XMPP notifier' to 'an XMPP notifier'
@MattGill98 MattGill98 deleted the PAYARA-1486-XMPP-notifier-Room-Id-Name branch June 18, 2018 08:59
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.

5 participants