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

The lazy loading dialog is making repeat appearances #23377

Closed
robintown opened this issue Sep 30, 2022 · 3 comments · Fixed by matrix-org/matrix-js-sdk#2718
Closed

The lazy loading dialog is making repeat appearances #23377

robintown opened this issue Sep 30, 2022 · 3 comments · Fixed by matrix-org/matrix-js-sdk#2718
Assignees
Labels
A-Lazy-Loading O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect X-Regression X-Release-Blocker

Comments

@robintown
Copy link
Member

Steps to reproduce

According to a user's report:

  1. Update to the latest develop.element.io
  2. Get the lazy loading dialog
  3. Press the OK button
  4. Element reloads
  5. goto 2

This is also reproducible by running any of our Cypress tests.

Outcome

What did you expect?

The lazy loading dialog should not appear

What happened instead?

cypress-screenshot(1)

Since the dialog appears repeatedly on each reload, this effectively blocks all interaction with the app.

Operating system

NixOS unstable

Browser information

Chromium 105.0.5195.125

URL for webapp

The develop branch

Application version

Element version: 8e841be-react-7a33818bd7ec-js-f398e3564d14 Olm version: 3.2.12

Homeserver

No response

Will you send logs?

No

@robintown robintown added T-Defect X-Regression S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Release-Blocker A-Lazy-Loading O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Sep 30, 2022
@robintown
Copy link
Member Author

matrix-org/matrix-js-sdk#2713 likely related

@robintown
Copy link
Member Author

Tentative fix:

diff --git a/src/store/indexeddb-local-backend.ts b/src/store/indexeddb-local-backend.ts
index 0ab50db35..21166f575 100644
--- a/src/store/indexeddb-local-backend.ts
+++ b/src/store/indexeddb-local-backend.ts
@@ -167,6 +167,10 @@ export class LocalIndexedDBStoreBackend implements IIndexedDBBackend {
             logger.log(
                 `LocalIndexedDBStoreBackend.connect: upgrading from ${oldVersion}`,
             );
+            if (oldVersion < 1) {
+                // The database did not previously exist
+                this._isNewlyCreated = true;
+            }
             DB_MIGRATIONS.forEach((migration, index) => {
                 if (oldVersion <= index) migration(db);
             });

@t3chguy
Copy link
Member

t3chguy commented Sep 30, 2022

Regressed by matrix-org/matrix-js-sdk#2713

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Lazy-Loading O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect X-Regression X-Release-Blocker
Projects
None yet
2 participants