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

Allow manually specifying the snapshot URL #1409

Closed
wants to merge 2 commits into from
Closed

Allow manually specifying the snapshot URL #1409

wants to merge 2 commits into from

Conversation

frivoal
Copy link
Contributor

@frivoal frivoal commented Dec 13, 2018

This comes handy if you want to use bikeshed with one of the snapshotStatues to produce something that looks exactly like one of the things that goes on TR, without actually going on TR. For instance, the Process (or the patent policy, or any number of official dated documents that do or should have an ED as well).

In addition, creating a new status (accessible to anyone?) within datedStatuses group would also be useful, so that I didn't have to use header.include to suppress the insertion of <h2>[LONGSTATUS]</h2> would also be nice, but:

  • That's an addition, not an alternative to this patch, so you should still take this in
  • for now, I can indeed use header.include and to abuse another Status (like REC), so that's not a pressing need
  • Creating a new status means I need to pick a name for it, and naming is hard :)

For datedStatuses, when Snapshot key has not been specified,
"This Version" and "Editor's Draft" would point to the same place.
Only include one of the two, and prefer "This Version" for compatibility
with the behavior prior to the introduction of the Snapshot key.

Side note: checking for "not 'version' in mac" is not strictly
necessary, given that when the status is one of datedStatuses,
'version' will always be in mac. But while that's true in current code,
future evolutions might change that, and better be safe than sorry.
@frivoal
Copy link
Contributor Author

frivoal commented Jan 28, 2019

I still need this. Can you let me know if it's ok, or if I need to change something?

@tabatkins
Copy link
Collaborator

I was concerned about the possible effect this might have on WHATWG documents, since I think it would start hitting them. But I haven't tested yet, so I'm not sure. Will review more thoroly soon.

@frivoal
Copy link
Contributor Author

frivoal commented Jun 24, 2020

Ping. Process 2020 and Patent Policy 2020 are wrapping up, and I'll be needing this very soon.

@frivoal
Copy link
Contributor Author

frivoal commented Oct 7, 2021

@tabatkins This feature would come in really handy, can you please make it work? I'm willing to help, but only if you'll land it soon after I try, cause I can't maintain long-lived pull requests.

Briefly, the spec is to be able to generate something like this: https://www.w3.org/2020/Process-20200915/

  • A manually provided url for the stylesheet (so that I can point it to the base w3c stylesheet)
  • A Date
  • A custom (possibly empty) string, for usage in the the #profile-and-date element before the date
  • both an "ED" link and a "This version" link
  • Provide the "This version" link manually, don't derive it from the date
  • Let me do that from any group

This pull request was closed.
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.

2 participants