bustikiller/wedding-planner
1
0
Fork 0
Code Issues Pull Requests 4 Actions Packages Projects Releases Wiki Activity

Define CUD endpoints for the Groups model #173

Merged
bustikiller merged 3 commits from groups-endpoints into main 2024-12-08 10:53:00 +00:00
Conversation 0 Commits 3 Files Changed 11 +133 -32
bustikiller commented 2024-12-08 10:40:07 +00:00
Owner
Copy Link

The class Groups::SummaryQuery was not filtering groups by wedding since ActsAsTenant scopes don't apply to raw SQL queries. This query object has been rewritten in a way it leverages the gem's scopes.

Additionally, we've included endpoints to create, update and delete groups.

The Guest model has been updated to allow null groups IDs (when a group has been removed, or a user is still ungrouped)

The class `Groups::SummaryQuery` was not filtering groups by wedding since `ActsAsTenant` scopes don't apply to raw SQL queries. This query object has been rewritten in a way it leverages the gem's scopes. Additionally, we've included endpoints to create, update and delete groups. The `Guest` model has been updated to allow null groups IDs (when a group has been removed, or a user is still ungrouped)
bustikiller added 2 commits 2024-12-08 10:40:07 +00:00
Fix summary query to leverage ActsAsTenant scopes
All checks were successful
Check usage of free licenses / check-licenses (pull_request) Successful in 1m10s
Details
Add copyright notice / copyright_notice (pull_request) Successful in 1m35s
Details
Run unit tests / unit_tests (pull_request) Successful in 5m11s
Details
20127398c6
bustikiller added 1 commit 2024-12-08 10:41:27 +00:00
Add copyright notice
All checks were successful
Check usage of free licenses / check-licenses (pull_request) Successful in 2m37s
Details
Add copyright notice / copyright_notice (pull_request) Successful in 3m11s
Details
Run unit tests / unit_tests (pull_request) Successful in 4m23s
Details
9f0773647f
bustikiller changed title from groups-endpoints to Define CUD endpoints for the Groups model 2024-12-08 10:41:45 +00:00
bustikiller merged commit cbcb7b70e3 into main 2024-12-08 10:53:00 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bustikiller/wedding-planner#173
No description provided.
Delete Branch "groups-endpoints"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?