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

Remove obsolete SharedStorage interfaces #25450

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

Elchi3
Copy link
Member

@Elchi3 Elchi3 commented Dec 17, 2024

The SharedStorage API has been added manually in Oct 2023: #20375
The content side seems to have landed in December 2023: mdn/content#30427

However, in January 2024 and in following months, there seem to have been at least two major spec changes that have not been taken into account in BCD or in docs:

These spec updates remove a number of interfaces and I also don't see them in the current Chrome.
They either need to be removed completely (done in this PR), or we need a version_removed.
Also, I think the docs should probably be updated for this, too. cc @chrisdavidmills

@Elchi3 Elchi3 added the needs content update This PR needs a corresponding update to mdn/content to update the documentation label Dec 17, 2024
@github-actions github-actions bot added data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API size:l [PR only] 101-1000 LoC changed labels Dec 17, 2024
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

LGTM, I'm in favor of removing.

@caugner caugner merged commit 5083197 into mdn:main Dec 19, 2024
8 checks passed
@mdn-bot mdn-bot mentioned this pull request Dec 19, 2024
@Elchi3 Elchi3 deleted the sharedstorage-api-removals branch December 19, 2024 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API needs content update This PR needs a corresponding update to mdn/content to update the documentation size:l [PR only] 101-1000 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants