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

Handle setting and updating an overarching goal #38

Merged
merged 4 commits into from
Oct 9, 2024
Merged

Conversation

jhodapp
Copy link
Member

@jhodapp jhodapp commented Oct 6, 2024

Description

This PR sets and updates an overarching goal for this coaching session.

Note: pairs with this backend PR

GitHub Issue: None

Changes

  • Adds a new OverarchingGoalContainer component that contains the OverarchingGoal, the AgreementList and the ActionList components
  • Adds a new OverarchingGoal component that handles setting/updating an overarching goal

Screenshots / Videos Showing UI Changes (if applicable)

Drawer closed:
Jim __ Caleb _ Oct 10_ 2024

Drawer opened:
Jim __ Caleb _ Oct 10_ 2024

Testing Strategy

  1. Navigate to a coaching session page
  2. Click the down chevron button to expand the drawer
  3. You should be able to click in the overarching goal line into a textbox and type a new goal
  4. Either click "Set" or press with the focus in the textbox
  5. Notice that the drawer closes and the overarching goal label should be updated with whatever you entered.

Concerns

None

@jhodapp jhodapp added enhancement Improves existing functionality or feature feature work Specifically implementing a new feature labels Oct 6, 2024
@jhodapp jhodapp self-assigned this Oct 6, 2024
@jhodapp jhodapp marked this pull request as ready for review October 9, 2024 15:58
@jhodapp jhodapp merged commit fdd6406 into main Oct 9, 2024
1 check passed
@jhodapp jhodapp deleted the overarching-goals branch October 9, 2024 20:11
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