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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

edhelas
Copy link
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
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
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
Member

Kev commented Oct 30, 2023

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

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.

None yet

4 participants