Add Category Enum and Refactor #174
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Only merge after DB is prepared, i.e. all questions purged or updated to new format. |
SelwynAng
approved these changes
Oct 19, 2024
…425S1/cs3219-ay2425s1-project-g50 into enhancement/question-cat-enum
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #145, #159
Overview
To allow us to have finite categories for matching, we make use of enums.
Complexity changes
easy->Easymedium->MediumHard->HardCategory changes
No longer free text, but string enum instead, the following are current possible values:
AlgorithmsArraysBit ManipulationBrainteaserDatabasesData StructuresRecursionStringsOther changes
On top of the above, major refactoring of
frontendandquestion-servicewas done to improve code quality.