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

Added sentry logging to local revisions service for localStorage errors #21078

Merged
merged 6 commits into from
Sep 25, 2024

Conversation

cmraible
Copy link
Collaborator

@cmraible cmraible commented Sep 23, 2024

no issue

  • Added Sentry logs to capture how often we are running into QuotaExceededErrors when saving local revisions to localStorage, to help in deciding if localStorage is sufficient, or if we need to expand to e.g. IndexedDB.
  • Also adds some handling to ignore errors when calling localStorage.setItem() elsewhere in the admin app to avoid crashing if localStorage isn't supported or the quota is exceeded.

@cmraible cmraible marked this pull request as ready for review September 24, 2024 00:25
@cmraible cmraible merged commit 0125f52 into TryGhost:main Sep 25, 2024
35 checks passed
vlavrynovych added a commit to vlavrynovych/Ghost that referenced this pull request Sep 25, 2024
* main: (411 commits)
  Shipped comments-ui@0.18.2
  Fixed various visual details in Comments UI (TryGhost#21121)
  Shipped comments-ui@0.18.1
  Update dependency mysql2 to v3.11.3 [SECURITY]
  Added reply button to comment replies
  Update dependency mysql2 to v3.11.2 [SECURITY]
  Fixed flaky member expertise test
  Update dependency mysql2 to v3.11.1 [SECURITY]
  Increased parallelisation of Nx processes
  Update tiptap monorepo to v2.7.2
  Fixed Article feature image not showing up in the feed
  Fixed requiring passing tests for canary builds
  Updated Nx to v19
  🐛 Fixed infinite loops in `setFeatureImageCaption` for deleted posts (TryGhost#21081)
  Fixed versioning and npm visibility for internal apps
  Update dependency html-validate to v8.24.0
  Implemented lazy loading for Inbox & Activities (TryGhost#21106)
  Added sentry logging to local revisions service for localStorage errors (TryGhost#21078)
  Added posts restore UI (TryGhost#21096)
  Fixed bounce rate on stats page (TryGhost#21097)
  ...

# Conflicts:
#	ghost/i18n/locales/uk/comments.json
#	ghost/i18n/locales/uk/portal.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant