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

Release/v2.5.2 [main] #558

Merged
merged 5 commits into from
Nov 27, 2024
Merged

Release/v2.5.2 [main] #558

merged 5 commits into from
Nov 27, 2024

Conversation

mbanting
Copy link
Contributor

Merge release/v2.5.2 to main

sultanofcardio and others added 5 commits November 15, 2024 12:09
…e activity/flow using "Take Now" (M2-8219) (#552)

This ticket fixes an incorrect context access that resulted in an undefined dereference (`applet.id`)
…2-8164, M2-8026) (#547)

* fix: normalize self-report targetSubjectID

In the web app, self-report assessments are identified and tracked using
`targetSubjectId=null`. However, for past submissions, the BE returns
the raw value of `targetSubjectId`, which is the subject ID of the
respondent. These submissions needed to be normalized to have their
`targetSubjectId` set to `null`, but to do so, needed to find out what
the subject ID of the respondent (currently logged in user) is.

So added a new `respondentMeta` property to the applet `base_info`
endpoint on the BE, so that this value is now available to the FE to
perform the needed equality comparison against `targetSubjectId`.

* fix: use local time zone for existing submissions

Depending on your time zone and the time of day, the date returned by
the submissions endpoint could have been parsed incorrectly using the
previous approach. As a result, previous same-day submissions were
being interpreted as having occurred on the previous day and resulting
in daily activities showing when they shouldn't be.

Fixed this using a simpler and time zone-safe parsing method.
)

* feat: implement content truncation for Action Plan cards
* chore: switch time formatting to use existing utility function
* chore: replace `<br />` with 32px height box
* fix: missing scaling calculation for max page height
* chore: apply scaling to page min height
* feat: add Action Plan support for `paragraphText`

---------

Co-authored-by: Billie He <[email protected]>
Co-authored-by: farmerpaul <[email protected]>
* fix: update axios and jsdom versions in package.json to match required versions and lint newest required fixes (M2-7990)

* fix: using latest versions for axios and dompurify

* fix: adding chapel for allowing minor and patches updates on dompurify

* fix: updating yarn lock

---------

Co-authored-by: Ramir Mesquita <[email protected]>
@mbanting mbanting merged commit a7b1082 into main Nov 27, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants