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.

Sign up for GitHub

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

Jump to bottom

Implementation Suggestions section needs review #428

Closed
burnburn opened this issue Jan 12, 2017 · 8 comments
Closed

Implementation Suggestions section needs review #428

burnburn opened this issue Jan 12, 2017 · 8 comments
Assignees
@stefhak
Labels
List discussion needed

Comments

@burnburn
Copy link
Contributor

Once PR #395 is applied, all of section 10.6 (Implementation Suggestions) will clearly be marked as Non-normative. There is very old text in this section, some of which I suspect we all treat today as normative requirements, and other text which may no longer be relevant (such as the permissions text).

This text needs a complete review again now that it has all been marked non-normative.
@stefhak
Copy link
Contributor

stefhak commented Mar 13, 2017 •
edited
Loading

I am assigned to do this review, however after reading section 10.6 I realize it would probably be much better if someone involved in the implementation of these parts did the review. I would propose re-assigning this Issue.

@burnburn
Copy link
Contributor Author

@jan-ivar, @ShijunS , @alvestrand can you please review or assign reviewers from your orgs?

@jan-ivar
Copy link
Member

Lgtm. My concern with permissions was resolved, modulo w3c/permissions#131 (comment).

@ShijunS
Copy link
Contributor

ShijunS commented Mar 16, 2017

LGTM

@alvestrand
Copy link
Contributor

@guidou can you review this part of the spec? I'm simply not finding the round toit.

@guidou
Copy link
Contributor

guidou commented Mar 23, 2017

I'll review it.

@guidou
Copy link
Contributor

guidou commented Mar 23, 2017

lgtm

@stefhak
Copy link
Contributor

stefhak commented Mar 30, 2017

Reviewers are OK with the current version. Close?

@aboba aboba closed this as completed Mar 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stefhak stefhak

Labels
List discussion needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@alvestrand @aboba @burnburn @stefhak @jan-ivar @ShijunS @guidou