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

IDB freezes on extension updates #7846

Closed
twschiller opened this issue Mar 7, 2024 · 1 comment · Fixed by #7852
Closed

IDB freezes on extension updates #7846

twschiller opened this issue Mar 7, 2024 · 1 comment · Fixed by #7852
Assignees
Labels
bug Something isn't working infrastructure user experience Improve the user experience (UX)

Comments

@twschiller
Copy link
Contributor

twschiller commented Mar 7, 2024

Context

Discussion

  • In the past we've had problems where we weren't correctly closing IDB connections
  • We have an IDBErrorDisplay for use with ErrorBoundary. However, what's happening is the request just hangs, it doesn't error

Workaround

  • Workaround is to close/reopen Chrome. NOTE: reloading only the extension does not resolve the problem

Potentially Related Issues

Known IDB Issues

@twschiller twschiller added bug Something isn't working infrastructure labels Mar 7, 2024
@twschiller twschiller changed the title IDB freezes on extension upgrades IDB freezes on extension updates Mar 7, 2024
@twschiller
Copy link
Contributor Author

twschiller commented Mar 7, 2024

One idea we could implement is a hook/component that tries to perform a pageRegistry:count() call on IDB, and sets a flag to true if it doesn't get a response in a few seconds (and can clear the flag once it gets a response, or the call errors)

The UI can then show a modal or banner saying: "We're having trouble connecting your browser's local database, please close and reopen your browser.". It could also include a link to our troubleshooting docs: https://docs.pixiebrix.com/how-to/troubleshooting

Related Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working infrastructure user experience Improve the user experience (UX)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant