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

Minor issue in editors draft: Broken link to latest published #2582

Closed
plannero opened this issue Sep 13, 2023 · 5 comments
Closed

Minor issue in editors draft: Broken link to latest published #2582

plannero opened this issue Sep 13, 2023 · 5 comments
Labels
Spec-EAAMapping The issue affects the EPUB Accessibility - EU Accessibility Act Mapping WG Note

Comments

@plannero
Copy link

In the GitHub editors draft version of the ePub-EAA mapping (https://w3c.github.io/epub-specs/epub33/epub-a11y-eaa-mapping/) dated September 12 2023, the very first link on the page (to "Latest published version") has a URL ("https://www.w3.org/epub-a11y-eaa-mapping/") that leads to a 404 error message.

I guess it should be changed to "https://www.w3.org/TR/epub-a11y-eaa-mapping/".

@mattgarrish
Copy link
Member

Yes, the editor drafts for all the notes still have "epub" as the working group in the respec initialization metadata, but it's now "pm" (publishing maintenance working group). That's causing respec to default to its base profile and break the links.

Should we republish all the notes to update the group contact info, etc.? (Or turn off echinda and fix the editor drafts but leave republishing until we have more significant changes?)

Thoughts @iherman @wareid @dauwhe @shiestyle ?

@mattgarrish mattgarrish added the Spec-EAAMapping The issue affects the EPUB Accessibility - EU Accessibility Act Mapping WG Note label Sep 13, 2023
@iherman
Copy link
Member

iherman commented Sep 14, 2023

Should we republish all the notes to update the group contact info, etc.? (Or turn off echinda and fix the editor drafts but leave republishing until we have more significant changes?)

Thoughts @iherman @wareid @dauwhe @shiestyle ?

I believe this is painless, and mostly invisible to everyone, so it could be done.

That being said: I am surprised. The original issue, as raised by @plannero claims that the 'latest version' link did not include the '/TR/' in the URL, but I do not see this error. It looks good to me.

@plannero
Copy link
Author

plannero commented Sep 14, 2023

That's odd. "/TR/" is still missing for me. Not that it's a big problem, but I just wanted to document it when I stumbled on it.

image

@mattgarrish
Copy link
Member

It seems to depend on the browser. When I tried it in Edge yesterday I was going to post the same comment about it being fine, but when I tried in Firefox and Chrome the link was broken. In Edge I don't get the errors about respec defaulting to the base profile. Maybe we should report the issue in the respec tracker, too.

@mattgarrish
Copy link
Member

I'm going to close this issue as we fixed the problem when we republished all the notes in #2599.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Spec-EAAMapping The issue affects the EPUB Accessibility - EU Accessibility Act Mapping WG Note
Projects
None yet
Development

No branches or pull requests

3 participants