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 the OverarchingGoal.title to the CoachingSessionSelector component #60

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

jhodapp
Copy link
Member

@jhodapp jhodapp commented Dec 29, 2024

Description

This PR adds the OverarchingGoal.title to the CoachingSessionSelector component for more context in choosing the right session to join.

GitHub Issue: None

Changes

  • Adds the OverarchingGoal.title to the CoachingSessionSelector component
  • Adds a new OverarchingGoalStateStore
  • Adds SWR-based fetch hooks for working with OverarchingGoals

Screenshots / Videos Showing UI Changes (if applicable)

Screenshot 2024-12-29 at 15 43 25 Screenshot 2024-12-29 at 15 44 21

Testing Strategy

  1. Login
  2. Verify that the overarching goal for a session is display in the selector component
  3. If it isn't, "No goal set" should display instead

Concerns

None

…nt for more context in choosing the right session to join.
@jhodapp jhodapp added enhancement Improves existing functionality or feature feature work Specifically implementing a new feature labels Dec 29, 2024
@jhodapp jhodapp added this to the 1.0-rc2 milestone Dec 29, 2024
@jhodapp jhodapp requested a review from qafui December 29, 2024 21:45
@jhodapp jhodapp self-assigned this Dec 29, 2024
@jhodapp jhodapp merged commit e192903 into main Dec 30, 2024
4 checks passed
@jhodapp jhodapp deleted the display_overarching_goal_in_coaching_session_selector branch December 30, 2024 03:14
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.

2 participants