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

Complete Notification Page Functionality (Backend) #53

Merged
merged 7 commits into from
Feb 20, 2024
Merged

Complete Notification Page Functionality (Backend) #53

merged 7 commits into from
Feb 20, 2024

Conversation

seancfong
Copy link
Member

Authors:
@arralia @seancfong

What does this PR contain?

  • Route to approve user alters users table
  • Creating an account updates the user timestamp
  • Published schedules are created with the correct timestamp
  • Additional routes for querying published schedule

How did you test these changes?

Attach images (if applicable)

@ThatMegamind ThatMegamind linked an issue Feb 20, 2024 that may be closed by this pull request
Complete Notification Page Functionality (Backend) #45
Closed
ThatMegamind
ThatMegamind approved these changes Feb 20, 2024
View reviewed changes
Copy link
Collaborator

@ThatMegamind ThatMegamind left a comment

Choose a reason for hiding this comment

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

Had to fix a few merge conflicts, but otherwise looks good!

@ThatMegamind ThatMegamind merged commit 2652c4e into dev Feb 20, 2024
3 checks passed
@ThatMegamind ThatMegamind deleted the #45-notification-page-functionality branch February 20, 2024 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ThatMegamind ThatMegamind ThatMegamind approved these changes

@michellelin1 michellelin1 Awaiting requested review from michellelin1

@h0ethan04 h0ethan04 Awaiting requested review from h0ethan04

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Complete Notification Page Functionality (Backend)
3 participants
@seancfong @ThatMegamind @arralia