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

XEP-0045: Notification of role and affiliation changes #1358

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

guusdk
Copy link
Contributor

@guusdk guusdk commented Jul 5, 2024

The mechanisms in which occupants are notified of a role or affiliation change is quite similar throughout the specification, but is not consistently documented. The changes herein are an attempt to improve consistency.

The first two commits are rather small. They intend to make smaller improvements.

The last commit introduces a new section, and adds references from other sections that deal with sending notifications of role and/or affiliation changes.

guusdk added 3 commits July 5, 2024 19:32
…tions

A presence-based notification of a role or affiliation change can only be sent when the affected user is in the room. The XEP mostly used wording to this effect. In this commit, this wording has been applied to a few places where it was missing.
The mechanisms in which occupants are notified of a role or affiliation change is quite similar throughout the specification, but is not consistently documented.

This commit introduces a new section, describing the pre-existing MUST of using a presence stanza when to affected user is in the room, as well as the pre-existing MAY of usinga message stanza when the user is not.
@github-actions github-actions bot added the Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change. label Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant