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.

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

Jeffn/cohort plans #26

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

Jeffn/cohort plans #26

wants to merge 6 commits into from

Conversation

jnakaso
Copy link
Contributor

@jnakaso jnakaso commented Feb 17, 2025

Description

  • Support for adding plans to cohort
  • db schema updated accommodate deletion and cascading deletes.

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Related Tickets & Documents

https://das-culturousexchange-team.atlassian.net/browse/SCRUM-40

@jnakaso jnakaso requested a review from a team February 17, 2025 22:38
Copy link

github-actions bot commented Feb 17, 2025

Visit the preview URL for this PR (updated for commit 037f16c):

https://culturous-exchange--pr26-jeffn-cohort-plans-lgaolj4j.web.app

(expires Thu, 27 Feb 2025 15:15:21 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: adcc28caa9b7c1d3b1670a080fce5f1f054a3a92

@jnakaso jnakaso changed the base branch from main to dev February 17, 2025 22:48
Copy link
Contributor

@kssampson kssampson left a comment

Choose a reason for hiding this comment

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

Just noting: From what I can see, once this PR and the stepper PR are merged, Colin's student id to uuid PR should be the last one that syncs the tables using uuid with the correct references.

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.

2 participants