-
Notifications
You must be signed in to change notification settings - Fork 2
160 feature enhanced coaching sessions endpoint #202
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
Merged
jhodapp
merged 17 commits into
main
from
160-feature-enhanced-coaching-sessions-endpoint
Nov 10, 2025
Merged
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
dfa2700
Add user-scoped endpoints for organizations, actions, coaching sessio…
claude 061a031
Add enhanced coaching sessions endpoint with batch loading (Issue #160)
jhodapp 9d3bb71
Make sure response is included in lib.rs
jhodapp 2b400f4
Refactor coaching sessions endpoint for better architecture
jhodapp 502767e
Add some helpful code documentation for the different types and metho…
jhodapp 2d3df6a
Run cargo fmt
jhodapp b3d95b4
Fix doc-test for IncludeOptions usage example
jhodapp b17a6e8
Fix clippy derivable_impls warning for Status enum
jhodapp 43d1a81
Clean up bad rebase merge conflict from previous commit
jhodapp 51d5896
Reformat Status's display trait implementation to use human-friendly …
jhodapp 77d70dc
Don't duplicate sorting params from the web layer into the entity_api…
jhodapp b307fd3
Follow the established protect structure already laid out in web/src/…
jhodapp af16156
Use IndexParams directly in the action_controller.rs index method
jhodapp 6c51f7a
Move default sorting logic details outside of the coaching_session_co…
jhodapp 5b347e7
Use IndexParams directly in the overarching_goal_controller.rs index …
jhodapp 3ac7fbb
Remove unused code related to IndexParams
jhodapp ba2cb62
Ensure consistent user_id param handling
jhodapp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice