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

Update URLs for IndexedDB specifications #4406

Closed
wants to merge 1 commit into from

Conversation

mathewhodson
Copy link
Contributor

This should fix the duplicate entries in the specifications table on https://developer.mozilla.org/en-US/docs/Web/API/IndexedDB_API#specifications

image

@peterbe
Copy link
Contributor

peterbe commented Aug 4, 2021

@teoli2003 or @Elchi3 r?

@teoli2003
Copy link
Contributor

This is a bit tough:

  • we are retiring (little by little) the 3-column spec tables, and the use of SpecName and Spec2. Once this is done, we will be able to delete SpecData.json.
  • by doing this, we also get rid of mentions of obsolete specs, and of "numbered" links that get outdated.

We could get this in, but it is a bit of plaster on a wooden leg. We could also fix the table in the content to not use it.

So I'm torn. @Elchi3 ?

@Elchi3
Copy link
Member

Elchi3 commented Aug 5, 2021

I think we should update the content rather than linking to outdated specs.
If I see it correctly, all IndexedDB docs should always refer to https://w3c.github.io/IndexedDB/. The other IDB specs are not current and might lead people to read wrong material.

@teoli2003
Copy link
Contributor

teoli2003 commented Aug 5, 2021

Sounds good. @peterbe can you transfer this issue to mdn/content and assign it to me?

@peterbe
Copy link
Contributor

peterbe commented Aug 5, 2021

Sounds good. @peterbe can you transfer this issue to mdn/content and assign it to me?

This is a PR. They can't be transferred.

@teoli2003
Copy link
Contributor

Sorry @peterbe, of course it was a PR.

So basically, this file should be stale and any issue against it will be solved on the content side, until we can remove it.

This can be closed, as I opened a PR in mdn/content with the agreed fix.

@escattone escattone closed this Aug 9, 2021
@mathewhodson mathewhodson deleted the patch-2 branch August 10, 2021 14:34
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.

5 participants