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

Prepare status of the document for CR publication #1151

Merged
merged 5 commits into from
Nov 2, 2017
Merged

Prepare status of the document for CR publication #1151

merged 5 commits into from
Nov 2, 2017

Conversation

dontcallmedom
Copy link
Member

@dontcallmedom dontcallmedom commented May 2, 2017

Include features at risk
close #1084


Preview | Diff

@stefhak
Copy link
Contributor

stefhak commented May 4, 2017

Given the conversation at the VI May 2nd we should leave this unmerged for the time being.

Because Peer Identity is part of the requirements that the spec is expected to fulfill (and were agreed with the IETF RTCWEB Working Group), it shouldn't be removable without greater review process (which "at risk" would allow) - as such, it makes more sense not to mark it at risk (as discussed during virtual interim on May 2nd)
remove no longer relevant features at risk
note open substantive issues
link to draft implementation report
@aboba
Copy link
Contributor

aboba commented Oct 26, 2017

Is the intent to update this PR after feedback on the CR request?

…ocess

Based on feedback to transition request
@dontcallmedom dontcallmedom removed the request for review from fluffy October 30, 2017 14:10
Removing "This document will remain a Candidate Recommendation at least until 15 April 2018." as respec already adds a similar line:
"This Candidate Recommendation is expected to advance to Proposed Recommendation no earlier than 15 April 2018."
@stefhak
Copy link
Contributor

stefhak commented Nov 2, 2017

I leave it to @dontcallmedom to handle this PR!

@alvestrand
Copy link
Contributor

Now that CR publication is done, can we close this one as OBE?

@dontcallmedom
Copy link
Member Author

I think it would be usefully merged into the editors draft actually; but I don't mind very strongly if the editors prefer not to.

@aboba aboba merged commit a9984ed into master Nov 2, 2017
@vivienlacourba vivienlacourba deleted the cr-sotd branch March 2, 2018 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Note At-Risk features at front of document
6 participants