Repository navigation
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
|
There was a problem hiding this comment.
Just a quick question about the demo, is it expected for the default custom time to show 99 minutes? It seems odd that that would be the case. Also, it would be helpful if you provided the Figma link to the components you're implementing in the PR to help me quickly cross-check.
I've added the Figma link to the components (they should all be in that link). Also the 99 minutes was from my testing before that I forgot to change when recording the video. I have updated the video. |
caleb-bit
left a comment
There was a problem hiding this comment.
Could you split this PR up? 1000+ lines is generally a sign that the PR is too long.
|
Replaced by #135 |
Overview
Implemented UI of the reminder pages accessible from settings.
Changes Made
Added Reminder Option into Settings
Implemented UI for the Main Reminder Page
Implemented Class Reminders UI
Implemented Class Reminders View Model (data stored locally in DatastoreRepository.kt)
Edited Workout Reminders UI to save data after hitting done and deleted slider
Implemented Workout Reminders View Model (data stored locally in DatastoreRepository.kt)
Implemented navigation for all pages
Test Coverage
These features can be manually tested.
Navigate to Profile page, then navigate to Settings page
Click on Reminders option to access the 3 Reminder pages
Next Steps
Implement notifications for these reminders
Screenshots
Testing Video
Screen_recording_20261109_193840.mp4
https://www.figma.com/design/SJZ7i0N1Ly5PORfkJh7qXH/Uplift-SP26?node-id=6195-14550&t=iZS2IBggIzTXZNLn-1
Class Reminders UI from Figma
Workout Reminders UI from Figma