Skip to content

Draft: XEP-0372: urn:xmpp:reference:1#1281

Open
edhelas wants to merge 1 commit intoxsf:masterfrom
edhelas:feature/references-bump-type
Open

Draft: XEP-0372: urn:xmpp:reference:1#1281
edhelas wants to merge 1 commit intoxsf:masterfrom
edhelas:feature/references-bump-type

Conversation

@edhelas
Copy link
Copy Markdown
Contributor

@edhelas edhelas commented Apr 6, 2023

  • Redefine types
  • Bump the namespace.
  • Make uri parameter non mandatory
  • Remove the mentions chapter (defined in another XEP).
  • Add reference to XEP-0426
  • Add the XML schema
  • Add restrictions around the begin and end attributes
  • Complete the discovery section
  • Add the optional hreflang attribute

- Redefine types
- Bump the namespace.
- Make uri parameter non mandatory
- Remove the mentions chapter (defined in another XEP).
- Add reference to XEP-0426
- Add the XML schema
- Add restrictions around the begin and end attributes
- Complete the discovery section
- Add the optional hreflang attribute
@edhelas edhelas force-pushed the feature/references-bump-type branch from 0e3853a to dd2f16e Compare April 6, 2023 06:44
@edhelas edhelas changed the title XEP-0372: urn:xmpp:reference:1 Draft: XEP-0372: urn:xmpp:reference:1 Apr 6, 2023
@Kev Kev added the Needs Author The XEP is experimental and the PR was not made by the author. The author needs to acknowledge it. label Apr 24, 2023
@singpolyma
Copy link
Copy Markdown
Contributor

Why are mentions being removed? Isn't that most of the point of this XEP?

I'm also not sure we need to bump the namespace as this will break everyone using references today.

@Ppjet6
Copy link
Copy Markdown
Contributor

Ppjet6 commented Aug 20, 2023

The goal for it was to make the spec generic enough and without specific features, so that other specs could define their own types.

Mentions were supposed to be re-added with something like https://bouah.net/specs/mentions.html but I'm not going to pursue on this.

I guess personally I'd settle with just the following for the moment:

  • Add reference to XEP-0426
  • Add the XML schema
  • Add restrictions around the begin and end attributes
  • Add the optional hreflang attribute

I do think making it really generic is the most interesting part of the PR, but that also makes me wonder if the spec just isn't too broad.

Anyway this is still "Needs Author".

@Kev
Copy link
Copy Markdown
Member

Kev commented Oct 30, 2023

Hadn't clicked that this is waiting on my review, sorry. Have added it to my TODO.

@edhelas
Copy link
Copy Markdown
Contributor Author

edhelas commented Dec 2, 2025

Bump @Kev

Comment thread xep-0372.xml
from='balcony@channels.shakespeare.lit'
to='romeo@montegue.lit/30d3d8'>
<reference xmlns='urn:xmpp:reference:0'
type='data'
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should this be type='urn:xmpp:reference:data:1' as well? Also, does type HAVE to be a URI? Seems implied because of service discovery, but maybe worth making explicit on line 116?

@Kev
Copy link
Copy Markdown
Member

Kev commented Mar 31, 2026

Having finally had a proper look at this, I think this combines some probably sensible things, with some not (e.g. removing mentions doesn't seem to serve a purpose).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Author The XEP is experimental and the PR was not made by the author. The author needs to acknowledge it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants