Sprint3 settings #40
Open
Sprint3 settings #40
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
6 tasks
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Sprint
School Name Standardization and Settings Page ⚙️ - Will & Hansini #36
Who worked on this sprint?
Will and Hansini
Features Implemented
Implemented a function that standardizes school names into a consistent format
Implemented the basic outline for the settings page
New files created
src/app/ui/multi-select.tsx
src/app/lib/school_name_standardize.ts
src/app/settings/page.tsx
Existing files modified
src/app/page.tsx
Acceptance Criteria
School name standardization function exists in src/lib/, is exported, and handles common inconsistencies
Standardization function has been tested with sample data from Google Drive and correctly identifies matching schools with different name formats
Settings page is implemented and matches the Figma
UI matches Figma
Code runs locally without errors and builds successfully
Merge via PR
Testing: how did you test?
Tested the school name standardization by plugging in values from the sample data in google drive, and seeing if it produced the expected ouput.
We tested the settings page by displaying the page in local host and seeing if it produced the output we wanted it to.
Features Not Implemented/Incomplete
The map in the figma for the settings page
Screenshots:
https://github.com/user-attachments/assets/c208cd09-7532-4423-8e7f-84222fc3e1b2
Tag Dan and Shayne
@danglorioso @shaynesidman
closes #36