-
Notifications
You must be signed in to change notification settings - Fork 123
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-0284: Various fixes, and add myself as an author #904
base: master
Are you sure you want to change the base?
Conversation
@stpeter, we need an ACK from authors to accept the PR! I don't know the handles for the other authors, happy if somebody can chime in. |
bc41d2c
to
e3ddf97
Compare
Fyi, Joonas Govenius’s email address doesn’t exist on Google’s servers anymore, and Tom Pusateri’s rejects emails sent from my server, so I couldn’t reach them. |
These had been forgotten in ae8b1eb.
This has only been tested against the XEP examples, but it should be complete.
I’d say the addition of an XML Schema is important enough to warrant putting it back on the Standards Track.
It will be nice to update the XEP-0284, @linkmauve, @Ppjet6, have you progressed on it? |
I can take a look at this pull request. Not sure why email bounced but try again. |
@pusateri I believe it's the message to Joonas that bounced. Thanks for taking a look! |
Hi @linkmauve apparently this has been approved by at least one of the authors. Can you please rebase and squash this so I, editor, can merge this? Thank you cheers |
I’m working on an implementation of this XEP, and I’d like to push it forward towards Draft in not so long, being an author will let me do that properly.