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

Add CTAs column to Patients table #88

Merged
merged 1 commit into from
Jan 17, 2021
Merged

Add CTAs column to Patients table #88

merged 1 commit into from
Jan 17, 2021

Conversation

patrick-du
Copy link
Contributor

@patrick-du patrick-du commented Jan 16, 2021 •
edited
Loading

Notion ticket
Associated Frontend PR

Changes

  • Migration to add CTAS (Int) column to Patients table
  • Update patient models, query/mutation schemas, and resolvers

Testing

  • Ensure querying patients, addPatient mutation, and updatePatient mutation performs the expected behaviour

Checklist

Code review doc for reference

before opening PR

  • check notion ticket
  • run linter
  • go through file diff

filling out PR

  • descriptive title
  • update notion ticket link
  • fill out Changes
    • (optional) add inline comments in diff
  • fill out Testing
  • (optional) add Screenshots
  • assign yourself to the PR

after opening PR

  • link PR to notion ticket
  • move ticket to In PR
  • make sure build passes
  • ping @ps in #paramedics-dev

@patrick-du patrick-du self-assigned this Jan 16, 2021
@patrick-du patrick-du mentioned this pull request Jan 16, 2021
Open
15 tasks
@petorsong petorsong merged commit 9ab8380 into master Jan 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@petorsong petorsong petorsong approved these changes

Assignees

@patrick-du patrick-du

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@patrick-du @petorsong