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

feat: D5 to D7 #82

Merged
merged 314 commits into from
Nov 14, 2024
Merged

feat: D5 to D7 #82

merged 314 commits into from
Nov 14, 2024

Conversation

tituschewxj
Copy link

@tituschewxj tituschewxj commented Nov 13, 2024

Merges the changes from D5 to D7 to main

Note: remember to tag the commit after this is merged.

solomonng2001 and others added 28 commits November 13, 2024 21:32
…handling

fix: error handling and history pagination bug
Add RabbitMQ URL to environment variables in test.yml
Attempt at N4 to add JavaScript into Code Editor
…tatus-in-local-storage

fix: store session info in localstorage
…ate-delete

Additional Integration tests for question service
Fix the matched difficulties and topics so that it will be an OR if either is empty. Also fixes an issue with difficulties matching using wrong variable topics.
Copy link

@bensohh bensohh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tituschewxj tituschewxj merged commit d99ffbc into main Nov 14, 2024
6 checks passed
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.

4 participants