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
Improve patent text on notes #1446
Conversation
marcoscaceres
commented
Dec 14, 2017
- closes Upcoming update to WG notes patent policy text #1443
@deniak, let me know if this looks right. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marcoscaceres Except for the name of the status, that PR looks good.
src/w3c/headers.js
Outdated
@@ -634,11 +634,23 @@ export function run(conf, doc, cb) { | |||
conf.isPR = conf.specStatus === "PR"; | |||
conf.isPER = conf.specStatus === "PER"; | |||
conf.isMO = conf.specStatus === "MO"; | |||
conf.isNote = conf.specStatus === "NOTE"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at the documentation, shouldn't the status be WG-NOTE
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are correct, there is also FPWD-NOTE, it seems. I still don't know why we have "NOTE"? It seems to be "a thing" - it has an associated W3C Style? Should we kill that?
tests/spec/w3c/headers-spec.js
Outdated
it("gracefully handles missing wgPatentURI", async () => { | ||
const ops = makeStandardOps(); | ||
const newProps = { | ||
specStatus: "NOTE", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WG-NOTE
instead of NOTE
?
tests/spec/w3c/headers-spec.js
Outdated
const ops = makeStandardOps(); | ||
const newProps = { | ||
wgPatentURI: "https://www.w3.org/pp-impl/123456/status", | ||
specStatus: "NOTE", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WG-NOTE
instead of NOTE
?
533187f
to
ac2face
Compare
@@ -634,11 +634,23 @@ export function run(conf, doc, cb) { | |||
conf.isPR = conf.specStatus === "PR"; | |||
conf.isPER = conf.specStatus === "PER"; | |||
conf.isMO = conf.specStatus === "MO"; | |||
conf.isNote = ["FPWD-NOTE", "WG-NOTE"].includes(conf.specStatus); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@deniak, let me know if "FPWD-NOTE" applies or not. I'm was unsure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think that applies too!
Will release later today. |