fix: update create call - Githubissues

metrotranscom / doorway

component-based web framework for affordable housing management

Apache License 2.0

3 stars 1 forks source link

fix: update create call #702

Closed ColinBuyck closed 1 week ago

ColinBuyck commented 1 week ago

This PR addresses #656

[x] Addresses the issue in full
[ ] Addresses only certain aspects of the issue

Description

This PR updates the user create call to include the agreedOfTermsOfServices that wasn't previously being saved.

How Can This Be Tested/Reviewed?

Run this locally, create a public account, check the db that the agreedToTermsOfService is set to true for that user.

Author Checklist:

[x] Added QA notes to the issue with applicable URLs
[x] Reviewed in a desktop view
[ ] Reviewed in a mobile view
[ ] Reviewed considering accessibility
[ ] Added tests covering the changes
[ ] Made corresponding changes to the documentation
[ ] Ran yarn generate:client and/or created a migration when required

Review Process:

Read and understand the issue
Ensure the author has added QA notes
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates