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

Add User Update Endpoint #106

Closed
wants to merge 4 commits into from
Closed

Add User Update Endpoint #106

wants to merge 4 commits into from

Conversation

calebbourg
Copy link
Collaborator

@calebbourg calebbourg commented Mar 4, 2025

Description

This PR introduces a pattern for providing partial updates to database tables via our API.
By partial updates I mean that we are able to modify a subset of columns for any given table without having to provide values for the the other columns. The pattern is somewhat based on the pattern for filtering queries as introduced in #102

Note: This new endpoint assumes that the user being updated will always be the current user (as noted inline in comments). At some point we will likely want a user to be able to make updates to another user. We can decide how the code diverges at that point when we know more about how we want that to work.

Changes

  • Adds generic update function
  • Updates user updating to use new function
  • Adds PUT /users update endpoint
  • Fixes a small issue with the Rapidoc for login

Testing Strategy

I was able to test partially updating a user in Rapidoc

Screenshot 2025-03-04 at 1 18 55 PM

Coming Up

  • Adding New Users
  • Updating Coaching Sessions
  • Stricter validations

Verified

This commit was signed with the committer’s verified signature.
calebbourg Caleb Bourg

Verified

This commit was signed with the committer’s verified signature.
calebbourg Caleb Bourg

Verified

This commit was signed with the committer’s verified signature.
calebbourg Caleb Bourg

Verified

This commit was signed with the committer’s verified signature.
calebbourg Caleb Bourg
I had broken the user login part of rapidoc a while back. This fixes it
@calebbourg calebbourg self-assigned this Mar 4, 2025
@calebbourg calebbourg added the enhancement Improves existing functionality or feature label Mar 4, 2025
@calebbourg calebbourg added this to the 1.0-beta1 milestone Mar 4, 2025
@calebbourg calebbourg changed the title Updates and creation Add User Update Endpoint Mar 4, 2025
@calebbourg calebbourg marked this pull request as ready for review March 4, 2025 18:31
@calebbourg calebbourg closed this Mar 4, 2025
@calebbourg calebbourg deleted the updates_and_creation branch March 4, 2025 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves existing functionality or feature
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant