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: Presence sent to occupants of a destroyed room includes a <destroy/> element #1365

Merged
merged 3 commits into from
Aug 23, 2024

Conversation

guusdk
Copy link
Contributor

@guusdk guusdk commented Jul 13, 2024

Even if no reason or alternate venue is provided, a <destroy/> element is needed to distinguish between other presence unavailable.

…<destroy/>` element.

Even if no reason or alternate venue is provided, a `<destroy/>` element is needed to distinguish between other presence unavailable.
@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 13, 2024
@iNPUTmice
Copy link
Contributor

Council said the MUST is better changed to a SHOULD so it is backward compatible with implementations that don’t include it.

…<destroy/>` element.

Adding a note that refers to ambiguity in previous versions of the specification, as per Council feedback.
@guusdk
Copy link
Contributor Author

guusdk commented Aug 12, 2024

Further discussion in the Council MUC suggest that most (if not all) Council members also agree to retaining the MUST when a note is added to comment on compatibility with earlier versions of the specification. I've pushed such a note moments ago.

@iNPUTmice iNPUTmice merged commit 8f766dc into xsf:master Aug 23, 2024
1 check passed
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.

2 participants