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

Handle case when sessionStorage is blocked #9785

Merged
merged 1 commit into from
Mar 25, 2025

Conversation

eth3lbert
Copy link
Contributor

This PR attempts to retrieve session storage within a try catch block to handle cases where it is blocked.
This should resolve #9784.

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ bug This PR primarily fixes a reported issue labels Mar 25, 2025
@runspired runspired merged commit 5f06bd2 into emberjs:main Mar 25, 2025
29 of 34 checks passed
@eth3lbert eth3lbert deleted the fix-session-storage branch March 25, 2025 21:46
gitKrystan added a commit that referenced this pull request Apr 1, 2025
* chore: improve contributing guidance for api documentation (#9778)

* chore: cleanup deprecations (#9779)

* docs: update overviews with recent library improvements (#9777)

* docs: update overviews with recent library improvements

* make docs preview even easier

* update more docs

* begin adding ember docs

* add all the docs

* docs: enable canary previews

* chore: fix workflow indentation

* make setup more robust

* fix boolean check in yml

* yml: your mileage lags

* fix deploy

* docs: improve schema-record documentation (#9781)

* chore: improve schema-record documentation

* add more docs for fields

* chore: add class coverage to docs tests (#9782)

* docs: expand field docs (#9783)

* Handle case when sessionStorage is blocked (#9785)

* chore(config): migrate renovate config (#9793)

chore(config): migrate config .github/renovate.json

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency bun-types to v1.2.5 (#9789)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update ember-core (#9791)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency qunit-dom to ^3.4.0 (#9790)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: make warnMessageNoModelForKey safe in prod (#9885)

* Backport pnpm-lock.yaml changes

* Backport docs-viewer

* Fix docs test expectations

---------

Co-authored-by: Chris Thoburn <runspired@users.noreply.github.com>
Co-authored-by: eth3lbert <eth3lbert+dev@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎯 canary PR is targeting canary (default) 🏷️ bug This PR primarily fixes a reported issue
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Errors when all cookies and localStorage are disabled with version 5.3.12
2 participants