-
Notifications
You must be signed in to change notification settings - Fork 30
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
4624 Navigating between activities using the nav menu doesn't update the Activity Name #4630
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4630 +/- ##
==========================================
+ Coverage 85.62% 94.32% +8.69%
==========================================
Files 274 277 +3
Lines 8802 8811 +9
Branches 1773 1773
==========================================
+ Hits 7537 8311 +774
+ Misses 1158 476 -682
+ Partials 107 24 -83
... and 103 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
This deploy was cleaned up. |
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.
Bug fixed! Well done with the cypress test too😄
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.
Looks good!
Resolves #4624
Description
This fix displays the correct activity name (on Activity Overview page) when navigating between activities. Before the fix, the previously viewed activity name was persisting in the text box under "Activity name".
Significant changes or possible side effects
useEffect
function to theNameForm
component which listens for changes in the "name" value and updates the field in reach hooks form when there is a change (in this case when we navigate from one activity to another)Automated test cases written
Steps to manually verify this change
This pull request is ready to code review when
tests, refactor
screenreader, text scaling) OR an exemption is documented
This pull request is ready to test when
This pull request is ready to review when QA has
This pull request can be merged when