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

fix: update scheduling model #30

Merged
merged 8 commits into from
Nov 13, 2021
Merged

fix: update scheduling model #30

merged 8 commits into from
Nov 13, 2021

Conversation

annechung
Copy link
Contributor

@annechung annechung commented Nov 11, 2021

Notion ticket link

Scheduling Table Changes

Implementation description

  • updating category on the scheduling table to be a string array

Steps to test

What should reviewers focus on?

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@annechung annechung requested a review from Shehryar21 November 11, 2021 02:15
@annechung annechung changed the title Update category type fix: update category type Nov 11, 2021
@annechung annechung changed the title fix: update category type fix: update scheduling model Nov 11, 2021
@annechung annechung requested a review from a team November 11, 2021 06:04
@annechung annechung merged commit c09e9fa into main Nov 13, 2021
@annechung annechung deleted the scheduling-category branch November 13, 2021 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants