Skip to content

Release Candidate - Materials 1.4 - #449

Merged
kramduny-cps merged 29 commits into
mainfrom
development
Jul 24, 2026
Merged

Release Candidate - Materials 1.4#449
kramduny-cps merged 29 commits into
mainfrom
development

Conversation

@kramduny-cps

Copy link
Copy Markdown
Collaborator

afeescps and others added 19 commits July 3, 2026 09:44
…#415)

* feat(telemetry): Add shared Application Insights client for Materials UI. Initialise the web SDK once at startup from VITE_APPLICATIONINSIGHTS_CONNECTION_STRING and export shared track helpers.

* feat(telemetry): sample non-exceptions and tag cloud role

* feat(telemetry): fix sonar issue
* syntax corrections on PR pipeline

* testing parameter options

* typo correction

* reverting to original parameter structure
* feat(tracking): track user actions, roles and time in telemetry

- Emit completion-based UserAction events (Reclassified, Updated,
  Discarded, Renamed, Redacted, UpdatedAutomatically, OpenedInNewWindow)
  with materialId/category context; View-in-new-window fires on click
- Capture page and per-tab dwell time; stamp MSAL user role on all items
- Sample non-exceptions (default 20%) while keeping exceptions and
  actions; set cloud role name from env
- Normalise route names to strip case URN/id

* fix(tracking): send materialId as string in action telemetry

Numeric materialId was routed into customMeasurements by the SDK while
the reader/redact screens sent it as a string in customDimensions. Cast
to string at all numeric call sites so it's consistently a dimension.

* fix(materials-ui): resolve ambiguous JSX spacing in discard caption (S6772)

Move the trailing space out of the visually-hidden span into an explicit
{' '} so the caption spacing is unambiguous. Output unchanged.

* refactor(materials-ui): clarify route-normalisation helper in telemetry

Rename normalisePath to stripCaseIdsFromRoute and extract the two regexes
into named constants (CASE_URN_AND_ID, NUMERIC_ID_SEGMENT) for readability.
No behaviour change.
Co-authored-by: abdul-cps <abdu.tarafdar@cps.gov.uk>
…Comms tabs (#436)

* feat(relabelling): Re-label Search screen field names in Materials & Comms tabs

* test: use corrector selector value

* test: use corrector selector API
#442)

* FCT2-20521-ensure-that-category-labels-in-the-filters-section-can-be-changed-without-breaking-functionality

* FCT2-20521-ensure-that-category-labels-in-the-filters-section-can-be-changed-without-breaking-functionality
…ew and Redact (#441)

* fix(materials): align category filter labels with Review and Redact

* refactor(materials): simplify category filter labels and checked default
* FCT2-20540-improve-fe-linting-and-agree-with-fe-devs

* FCT2-20540-improve-fe-linting-and-agree-with-fe-devs-2
#447)

* FCT2-16304-review-redact-tab-redaction-user-alerts-redaction-exceptions-update-2

* FCT2-16304-review-redact-tab-redaction-user-alerts-redaction-exceptions-update-2
* display error msg for redacting large file

* refactor hard coded number to const
benrome-cps
benrome-cps previously approved these changes Jul 15, 2026
robmolloy-cps and others added 8 commits July 16, 2026 10:02
#455)

* FCT2-16304-review-redact-tab-redaction-user-alerts-redaction-exceptions-update-1-3

* FCT2-16304-review-redact-tab-redaction-user-alerts-redaction-exceptions-update-1-3
#456)

* FCT2-16304-review-redact-tab-redaction-user-alerts-redaction-exceptions-update-1-4

* FCT2-16304-review-redact-tab-redaction-user-alerts-redaction-exceptions-update-1-4
* JAWS fixes

* sonar fixes

* remoev useEffect for closing modal

* mock route in test
…#458)

Emit AppOpened once per app load and stamp MSAL localAccountId on telemetry
so weekly reports can count distinct users without PageVisitTime sampling bias.
Add private-beta-weekly-usage.kql (rolling 7d/30d windows, beta floor
2026-05-07) and unit tests for telemetry helpers.
@sonarqubecloud

Copy link
Copy Markdown

@kramduny-cps
kramduny-cps merged commit 0f2777e into main Jul 24, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

8 participants