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.

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

Refactor/states #198

Merged
merged 95 commits into from
Aug 6, 2024
Merged

Refactor/states #198

merged 95 commits into from
Aug 6, 2024

Conversation

diogotvf7
Copy link
Contributor

@diogotvf7 diogotvf7 commented Feb 11, 2024 •
edited by tomaspalma

Progress

  • Course Picker
  • Combobox
  • New states
  • Refactor choosing multiple options on the emojis
  • Storing chosen options on local storage
  • Refactor random fill to handle new multiple options changes
  • Refactor copy and paste option because of changes to the multiple options structure
  • Refactor export csv to handle new multiple options structure
  • Refactor Schedule component

@netlify Netlify
Copy link

netlify bot commented Feb 11, 2024 •
edited

✅ Deploy Preview for tts-fe-preview ready!

Name Link
🔨 Latest commit b3f6240
🔍 Latest deploy log https://app.netlify.com/sites/tts-fe-preview/deploys/66b28aa6fb7fc0000876860e
😎 Deploy Preview https://deploy-preview-198--tts-fe-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@diogotvf7 diogotvf7 linked an issue Mar 5, 2024 that may be closed by this pull request
States refactor #181
Closed
4 tasks
@diogotvf7 diogotvf7 mentioned this pull request Mar 5, 2024
Closed
@tomaspalma tomaspalma mentioned this pull request Jul 22, 2024
Closed
thePeras and others added 12 commits July 22, 2024 12:40
@thePeras
Merge pull request #223 from NIAEFEUP/refactor/remove-options-list
a768776 
Unify multipleOptions and optionsList
@diogotvf7 @tomaspalma
Conflicts on class selector
6b7bc39 
Co-authored-by: Tomás Palma <[email protected]>
@tomaspalma
fix: class item selected class only updates if not previewing
7985786
@tomaspalma
Revert "Move all context state to CombinedProvider"
e9a3444 
This reverts commit 902aeec.
@thePeras
Fixed selected teachers classes options
a832eed
@thePeras
Merge pull request #228 from NIAEFEUP/fix/classes-options-and-seleted…
92d6a6e 
…-teachers

Fix Classes Options and Selected teachers
added the updated about-us page since it disappeared from this branch
ac5c8a9
@tomaspalma
Merge branch 'refactor/states' into refactor/schedule
0b09dec
@thePeras
Merge pull request #227 from NIAEFEUP/refactor/schedule
0c074db 
Refactor/schedule
@thePeras
Merge pull request #229 from NIAEFEUP/about-us
eb15482 
updated about-us page since it disappeared from this branch
@tomaspalma
fix: class fetching on picked courses is without race conditions
2e9b0a7
@thePeras
Merge pull request #231 from NIAEFEUP/refactor/pickedcourses-class-fe…
5b886da 
…tching

Solve race conditions from class fething on picked courses
@tomaspalma tomaspalma marked this pull request as ready for review August 2, 2024 14:24
@thePeras @tomaspalma
Only deletions ✂️
06a894c 
Co-authored-by: Tomás Palma <[email protected]>
@tomaspalma tomaspalma requested a review from a team August 2, 2024 17:50
thePeras
thePeras approved these changes Aug 3, 2024
View reviewed changes
Copy link
Member

@thePeras thePeras left a comment

Choose a reason for hiding this comment

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

✅ 🛫

@tomaspalma tomaspalma merged commit 2f61382 into develop Aug 6, 2024
4 checks passed
@tomaspalma tomaspalma mentioned this pull request Aug 7, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thePeras thePeras thePeras approved these changes

@tomaspalma tomaspalma tomaspalma approved these changes

Assignees

@Process-ing Process-ing

@tomaspalma tomaspalma

@diogotvf7 diogotvf7

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

States refactor
5 participants
@diogotvf7 @thePeras @tomaspalma @Process-ing @clarapbsousa