Skip to content

Availabilities CRUD#735

Draft
v-pak wants to merge 6 commits intomainfrom
CHAOS-711-availabilities-CRUD
Draft

Availabilities CRUD#735
v-pak wants to merge 6 commits intomainfrom
CHAOS-711-availabilities-CRUD

Conversation

@v-pak
Copy link
Copy Markdown

@v-pak v-pak commented Apr 11, 2026

Still need to fix Auth with handlers, handing this off to @Plebbaroni as per conversation in dev session

  • 2 Postgres tables for storing availabilities
    • user_campaign_availabilities - for user_id and campaign_id
    • availability_slots - for the actual timeslots
    • Done in this way to avoid changing user_id and/or campaign_id for every timeslot when only one change is needed
  • Models that modify these tables
  • 2 Handlers calling the models
    • get - gets all the timeslots for a given user_id and campaign_id
    • update - given a set of availabilities, updates the database to match for a given user_id and campaign_id
  • "/api/v1/availabilities/:user_id/:campaign_id" route, implementing a get and a patch router for the corresponding handlers

@v-pak v-pak marked this pull request as draft April 11, 2026 07:06
@Plebbaroni Plebbaroni self-requested a review April 17, 2026 11:06
Copy link
Copy Markdown
Contributor

@Plebbaroni Plebbaroni left a comment

Choose a reason for hiding this comment

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

Good work! Not gonna merge just yet as I wanna do the handlers and wanna keep this as a WIP. TYSM for the work on this PR though! :D

@v-pak
Copy link
Copy Markdown
Author

v-pak commented Apr 17, 2026

Yep! Handlers are unfinished and need proper auth code rather than the current placeholders, so this should stay as a draft until that's complete.

Thank you for the review!

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