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

Tab switching loads outdated watchlist information #1413

Open
xinyis991105 opened this issue Nov 6, 2021 · 1 comment
Open

Tab switching loads outdated watchlist information #1413

xinyis991105 opened this issue Nov 6, 2021 · 1 comment

Comments

@xinyis991105
Copy link
Contributor

Your environment
Current master branch of Autolab on my local machine Mac OS

Steps To Reproduce

  1. Go to a course with an appropriate amount of student information, so that the metrics feature has something to work with in the first place.
  2. Select risk conditions that will generate new watchlist information. Save and wait until the new information is successfully calculated.
  3. Switch to the watchlist tab. New watchlist instances are expected to show up. Currently, the list remains empty.
  4. Do a browser refresh. The new watchlist instances should now be available. Contact/Resolve certain instances. Switch to Contacted/Resolved tabs. Contacted/Resolved instances are expected to show up. Currently, another browser refresh is needed for that to happen.

I'm suspecting it might be something to do with switching from uglifier to terser for JavaScript but further research is needed for this. Let me know if this does not apply to other environments.

Screenshots
If applicable, add screenshots to help explain your problem.

@victorhuangwq
Copy link
Contributor

I have tested this issue both locally and on autolab-dev.

Locally, the issue was definitely replicated as per described.
However, when tested on autolab-dev, the issue does not occur.

I suspect it is due to a concurrency issue (the database updating on local machine occurs slower than it's getting retrieved). As such, I think I will defer addressing this issue until we deploy our master onto the autolab-dev branch for testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants