Skip to content

PO-2337#1907

Open
max-holland wants to merge 25 commits intomasterfrom
PO-2337
Open

PO-2337#1907
max-holland wants to merge 25 commits intomasterfrom
PO-2337

Conversation

@max-holland
Copy link
Copy Markdown
Contributor

@max-holland max-holland commented Oct 27, 2025

Jira link

See PO-2337

Change description

  • Introduce content digest interceptor

Testing done

  • Captured sha in console.log from libraries

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@iamfrankiemoran iamfrankiemoran self-assigned this Oct 29, 2025
@iamfrankiemoran iamfrankiemoran marked this pull request as ready for review October 30, 2025 14:43
Copy link
Copy Markdown
Contributor

@Arnabsubedi233 Arnabsubedi233 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@louisbriggs louisbriggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove references to local libraries.
Update PR description.

@iamfrankiemoran
Copy link
Copy Markdown
Contributor

@louisbriggs once libraries have been merged and published we'll remove the local versions. We also believe this requires a BE ticket too @max-holland

@iamfrankiemoran iamfrankiemoran changed the title Po 2337 PO-2337 Jan 28, 2026
@iamfrankiemoran
Copy link
Copy Markdown
Contributor

Will need to wait for the following:

  1. Node Library and Common UI Library to be published
  2. Then PO-2715 to be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants