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

Pressing browser back button can get stuck on the Loading screen #1748

Closed
jmgrady opened this issue Oct 20, 2022 · 3 comments · Fixed by #2971
Closed

Pressing browser back button can get stuck on the Loading screen #1748

jmgrady opened this issue Oct 20, 2022 · 3 comments · Fixed by #2971
Assignees
Labels
bug Something isn't working frontend goal Size: S Small sized issue

Comments

@jmgrady
Copy link
Collaborator

jmgrady commented Oct 20, 2022

Describe the bug

Clicking the browser Back button after adding words in Data Entry can cause The Combine to get stuck on the Loading... screen (>5 minutes).

To Reproduce

Steps to reproduce the behavior:

  1. Open/create a project;
  2. Click Review Entries;
  3. Click Data Entry;
  4. Add some words to one of the domains;
  5. Before exiting the Data Entry form press the browser's Back button;
  6. Loading... screen is displayed.

Expected behavior

I expected to be taken back to the domain tree.

Environment:

  • OS: Android 12 & Ubuntu 22.04
  • Browser Version:Chrome 106 & Firefox 105
@jmgrady jmgrady added bug Something isn't working frontend labels Oct 20, 2022
@jmgrady
Copy link
Collaborator Author

jmgrady commented Oct 20, 2022

Once The Combine gets into this state, if you log out and log in again, when you go to enter data, only the top-level domains are shown.

@jmgrady
Copy link
Collaborator Author

jmgrady commented Oct 20, 2022

Both problems (perpetual loading and the missing domains) need tobe fixed by clearing the browser cache.

@jmgrady
Copy link
Collaborator Author

jmgrady commented Oct 20, 2022

The missing domain part of the problem cannot be reproduced in v1.0.7-master.2 which has the fix that moves the semantic domain info to the database.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend goal Size: S Small sized issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants