Skip to content

Conversation

@bwang-icf
Copy link
Contributor

Changes to allow for v3 token exchange to function properly

JIRA Ticket:
BB2-4376

What Does This PR Do?

Allows for v3 functionality to work on the sample clients. Also includes more instructions in the README-sdk dev document to facilitate a better CMS developer experience.

What Should Reviewers Watch For?

Any areas of breaking functionality or any places where the added instructions for local setup is unclear. It is not the most straightforward process as is.

Validation

Validated with local run of auth flow

What Security Implications Does This PR Have?

Please indicate if this PR does any of the following:

  • Adds any new software dependencies
  • Modifies any security controls
  • Adds new transmission or storage of data
  • Any other changes that could possibly affect security?
  • Yes, one or more of the above security implications apply. This PR must not be merged without the ISSO or team security engineer's approval.

@jimmyfagan jimmyfagan self-assigned this Jan 20, 2026
Comment on lines +147 to +148
yarn build
yarn build:types
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably wrap this with `blah` or ```blah``` to match formatting from other sections and make this easier to read.

bwang-icf and others added 2 commits January 21, 2026 14:44
Co-authored-by: jimmyfagan <90421499+jimmyfagan@users.noreply.github.com>
Copy link
Contributor

@jimmyfagan jimmyfagan left a comment

Choose a reason for hiding this comment

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

Worked for me, approved!

@bwang-icf bwang-icf merged commit b8a6227 into main Jan 22, 2026
4 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.

3 participants