Skip to content

Reminders Pages UI [1/2] Added Main and Fixed Workout & Capacity Reminder Screens - #133

Closed
clin0595 wants to merge 14 commits into
mainfrom
reminders-1-main
Closed

clin0595 wants to merge 14 commits into
mainfrom
reminders-1-main

Conversation

@clin0595

Copy link
Copy Markdown
Contributor

Overview

Implemented the Main Reminder page in the settings, edited the Workout Reminder Page and the Capacity Reminder page, and added navigation to them.

Changes Made

  • Added Reminder Option into Settings

  • Implemented UI for the Main Reminder Page

  • 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 Workout Reminders and Capacity Reminders

Test Coverage

These features can be manually tested.
Navigate to Profile page, then navigate to Settings page
Click on Reminders option to access the 2 Reminder pages

Next Steps (delete if not applicable)

The Class Reminders Screen will be implemented in the next PR of this stack.

Related PRs or Issues

Reminders Pages UI [2/2] Added Class Reminders Screen (link here later)

Screenshots

Workout Reminders Screen and Capacity Reminders Screen
Screen_recording_20261110_163223.mp4

https://www.figma.com/design/SJZ7i0N1Ly5PORfkJh7qXH/Uplift-SP26?node-id=6195-14550&t=iZS2IBggIzTXZNLn-1

Workout Reminders UI from Figma Set Goals - End (1)

…actored localized SettingsOption component within SettingsScreen.kt to be a globalized PageOption component.
…navigation to ClassReminder page and Workout Page.
…kt, which accepts a string as the title. Implemented ClassReminder UI (untoggled).
# Conflicts:
#	app/src/main/java/com/cornellappdev/uplift/ui/MainNavigationWrapper.kt
#	app/src/main/java/com/cornellappdev/uplift/ui/components/reminders/goalsetting/GoalSettingHeader.kt
#	app/src/main/java/com/cornellappdev/uplift/ui/screens/reminders/WorkoutRemindersScreen.kt
@coderabbitai

coderabbitai Bot commented Nov 10, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a340f7e3-cc48-4bc6-8986-4c263e17408b
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@clin0595 clin0595 closed this Nov 10, 2026
@clin0595

Copy link
Copy Markdown
Contributor Author

Replaced by #134 to rename branch

@clin0595
clin0595 deleted the reminders-1-main branch November 10, 2026 22:02
@clin0595
clin0595 restored the reminders-1-main branch November 10, 2026 22:04
@clin0595
clin0595 deleted the reminders-1-main branch November 10, 2026 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant