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

Fix the ED links for Encrypted Media and Touch Events #403

Closed
wants to merge 2 commits into from

Conversation

foolip
Copy link
Collaborator

@foolip foolip commented Oct 3, 2017

"service-workers-1": {
"edDraft": { "replaceWith": "https://w3c.github.io/ServiceWorker/" }
},
"touch-events": {
"edDraft": { "replaceWith": "https://w3c.github.io/uievents/" }
Copy link
Owner

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops, of course!

@tobie
Copy link
Owner

tobie commented Oct 3, 2017

So just FYI, the downside of using overwrites is that you need to manually fix those if the data changes.

@foolip
Copy link
Collaborator Author

foolip commented Oct 3, 2017

So just FYI, the downside of using overwrites is that you need to manually fix those if the data changes.

Do you mean if the edDraft value changes in tr.rdf, or if the name of the entry as a whole changes? I could survive either. The difficulty of patching tr.rdf makes me do it this way.

@sideshowbarker, is this a problem that could be resolved W3C-side by putting tr.rdf on GitHub? I've sent patches by email, but it's just too much overhead to keep track of.

@tobie
Copy link
Owner

tobie commented Oct 3, 2017

Do you mean if the edDraft value changes in tr.rdf, or if the name of the entry as a whole changes? I could survive either.

Both.

The difficulty of patching tr.rdf makes me do it this way.

Yeah, that's basically just shifting the maintenance burden to Specref (i.e. to me), so I tend to favor upstreaming stuff.

@sideshowbarker, is this a problem that could be resolved W3C-side by putting tr.rdf on GitHub? I've sent patches by email, but it's just too much overhead to keep track of.

iirc @deniak is staff-contact for this stuff. That file is generated from a DB, so I don't think putting it on GitHub is technically an option.

@foolip
Copy link
Collaborator Author

foolip commented Oct 3, 2017

@deniak, could you have the things in https://github.com/tobie/specref/blob/master/overwrites/w3c.json fixed in tr.rdf, as well as the things in this PR?

@sideshowbarker
Copy link
Collaborator

@sideshowbarker, is this a problem that could be resolved W3C-side by putting tr.rdf on GitHub? I've sent patches by email, but it's just too much overhead to keep track of.

iirc @deniak is staff-contact for this stuff. That file is generated from a DB, so I don't think putting it on GitHub is technically an option.

Right, it’s a generated file and while I’m not sure @deniak is responsible for the system that generates it, I am sure he can speak from knowledge about what needs to be done to get it fixed/updated.

@deniak
Copy link
Collaborator

deniak commented Oct 4, 2017

I updated the editor's drafts for Encrypted Media, Touch Events, Web Crypto API, Service Workers 1, UI Events and Vribration in tr.rdf.
@foolip I'll deal with the ones you mentioned in the email you sent in a bit and will reply when it's done.

@foolip
Copy link
Collaborator Author

foolip commented Oct 4, 2017

Thanks @deniak! Even if tr.rdf itself can't be put on GitHub, maybe there could be an issue tracker for things like this? Or tr.rdf is checked in to a repo so that people can suggest changes but need not care about how tr.rdf is generated. The issues could be closed when the published version matches.

Closing this, thanks all!

@foolip foolip closed this Oct 4, 2017
@foolip foolip deleted the more-drafts branch October 4, 2017 06:58
foolip added a commit to foolip/day-to-day that referenced this pull request Oct 4, 2017
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.

4 participants