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

Course elements persistence #899

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TroyG-11
Copy link

Issue
closes #763

Database Changes/Migrations
No database schema changes were done for this fix.

Test Modifications
Added tests to verify that the schedule persists when navigating away from and back to the schedule page.

Test Procedure

Checkout the branch containing this commit.
Start the application locally (http://localhost:8080/).
On the schedule page, verify that your selected courses and schedule details are displayed.
Navigate to another route (e.g., http://localhost:8080/explore).
Return to the schedule page (http://localhost:8080/) and confirm that the schedule still displays correctly without needing a full page refresh.
Verify that UI states (e.g., course selections and class details) persist as expected.
Photos
Before: https://gyazo.com/fef9ac4e0f0ccf5235803bb58380eac1
After: https://gyazo.com/95b9549c07d4ac48529cebb58a432563

Additional Info
Noticed this issue occured when you weren't logged in.
Added an activated() lifecycle hook to the scheduler component to reload the schedule when the component is re-activated (i.e., when using ).
This fix addresses an issue where switching routes caused the schedule’s CSS classes to disappear until a manual refresh occurred.
Commit details: c3d8f4565885dc4d247c084ea2dd0132b7cd91af

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

Successfully merging this pull request may close these issues.

Bug — Classes don't show after going back to Schedule Page
1 participant