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.

Sign up for GitHub

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

Jump to bottom

Release/v2.5.1 [main] #555

Merged
merged 8 commits into from
Nov 21, 2024
Merged

Release/v2.5.1 [main] #555

merged 8 commits into from
Nov 21, 2024

Conversation

ChaconC
Copy link
Contributor

@ChaconC ChaconC commented Nov 20, 2024

Merge Release/v2.5.1 to main

LashaunnaS and others added 8 commits October 21, 2024 14:02
@LashaunnaS
Fix to the useSessionBanners hook to ensure that logic such as banner…
7b6c170 
… removal only occurs after authorization has changed. Replacing the useEffect with useMemo should ensure that any side effects that get triggered are due the state of the application once any further auth logic has been resolved.
@LashaunnaS
Store the original url so that when the app logs out the previously a…
f2d04c8 
…uthenticated user, the newly authenticated user will get redirected to the route originally specified.
@LashaunnaS
Capture possibility of of user coming from the loginroute once authen…
8aa8fe0 
…ticated.
@LashaunnaS
Remove use of useEffect hook. Which seems to be causing removeAllbann…
63045fd 
…ers to triggering after rerendering.
@ChaconC
fix: remove hardcoded strings M2-8179 (#549)
1688376
@AlejandroCoronadoN
feature: paragraph ItemResponseType for phrase builder response optio…
e6fb19e 
…ns WEB (M2-7762) (#542)

* Added new phrasalTemplate compatible reponse type
* Enabling breaklines when displaying paragraphs with multiple breaklinesa cross the text
@mbanting
Merge pull request #539 from ChildMindInstitute/fix/M2-8029-web-take-…
de90b96 
…now-bug-when-credentials-dont-match

fix: Warning banner & post-login redirect both fail when starting `Take Now` while already logged into Web App as wrong respondent (M2-8029)
@sultanofcardio @ChaconC
fix: The "Return to Admin App" button is not working when starting th…
ef997eb 
…e activity/flow using "Take Now" (M2-8219) (#552)

This ticket fixes an incorrect context access that resulted in an undefined dereference (`applet.id`)
@ChaconC ChaconC requested a review from mbanting November 20, 2024 15:04
@ChaconC ChaconC merged commit 908a21e into main Nov 21, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mbanting mbanting mbanting approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ChaconC @mbanting @LashaunnaS @AlejandroCoronadoN @sultanofcardio