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

Fetch coaching sessions for selecting the coaching session to join #22

Merged
merged 5 commits into from
Jul 2, 2024

Conversation

jhodapp
Copy link
Member

@jhodapp jhodapp commented Jun 29, 2024

Description

Adds a new CoachingSession type to reflect the backend Rust type and fetches 2 months (1 back, 1 forward) worth of sessions and display in the dashboard's select component. Allows the user to select a session to join.

Note: This PR requires the following backend changes to function correctly.

GitHub Issue: N/A

Changes

  • Adds a new CoachingSession type that reflects the backend Rust type
  • Fetches 2 months (1 back, 1 forward from now) worth of sessions
  • Display these sessions in the select component on the dashboard page
  • Allows the user to select and then join a valid coaching session (navigates to that page)

Screenshots / Videos Showing UI Changes (if applicable)

Dashboard with coaching sessions displayed:
Screenshot 2024-06-30 at 20 59 26

Coaching sessions page with endpoint shown in browser address bar:
Screenshot 2024-06-30 at 21 00 47

Testing Strategy

  1. Log in and the dashboard will show
  2. Select an organization
  3. Notice that one list of coaching relationships loads
  4. Select a coaching relationship
  5. Notice that a list of coaching sessions loads, or if there aren't any, "None found" displays
  6. With a valid coaching session chosen, notice that the Join button enables
  7. Click the Join button and notice that you're navigated to the /coaching-sessions/<uuid> and the coaching session page renders

Concerns

  • Purposefully keeping this PR simpler by not storing all of the dashboard's chosen UUIDs in a Zustand store. This will come in a future PR.

…fetches 2 months (1 back, 1 forward) worth of sessions and display in the dashboard's select component
@jhodapp jhodapp added enhancement Improves existing functionality or feature feature work Specifically implementing a new feature labels Jun 29, 2024
@jhodapp jhodapp self-assigned this Jun 29, 2024
@jhodapp jhodapp marked this pull request as ready for review July 1, 2024 02:09
@jhodapp jhodapp requested review from qafui and calebbourg July 1, 2024 02:10
Copy link
Collaborator

@qafui qafui left a comment

Choose a reason for hiding this comment

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

Nice Work!

@jhodapp jhodapp merged commit 759c8d1 into main Jul 2, 2024
1 check passed
@jhodapp jhodapp deleted the fetch_coaching_sessions branch July 2, 2024 16:13
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 feature work Specifically implementing a new feature
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants