Skip to content

Fix: Reopening editors #58

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Fix: Reopening editors #58

wants to merge 8 commits into from

Conversation

shirsho-12
Copy link
Contributor

Closes the following:

@shirsho-12
Copy link
Contributor Author

Note to reviewer: run and test for possible side effects

@heyzec
Copy link
Collaborator

heyzec commented Jul 6, 2025

Login via NUS seems to break with this error on this branch.
image
Can reproduce this on your end to check if im not tripping?

@heyzec
Copy link
Collaborator

heyzec commented Jul 6, 2025

It seems that the root cause of the above error is that the frontend has isVscode set to false. Hence, login, MCQ and editor is breaking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shirsho-12 @heyzec