Skip to content

Improved Raft over XMPP ProtoXEP#26

Closed
pmembrey wants to merge 4 commits intoxsf:masterfrom
pmembrey:master
Closed

Improved Raft over XMPP ProtoXEP#26
pmembrey wants to merge 4 commits intoxsf:masterfrom
pmembrey:master

Conversation

@pmembrey
Copy link
Contributor

Hi,

This PR addresses a couple of issues highlighted by the Council:

  • Ideally XEP shouldn't use Camel Case
  • Added a section on the justification for using rather than stanzas

Thanks!

Cheers,

Pete

inbox/raft.xml Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

+1 for providing a rationale using messages instead of IQ. But it should be "request-response", instead of "request/response".

@pmembrey
Copy link
Contributor Author

Updated ProtoXEP with feedback.

@linuxwolf
Copy link
Contributor

The Editor will get the update published shortly. Note that we will collapse the branch down, add a <revision/> entry for that update, and announce with an email that will look a lot like the first one.

@linuxwolf
Copy link
Contributor

Merged as 87e19dd.

@linuxwolf linuxwolf closed this Jul 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants