Skip to content

Comments

SWI-9652 Update SDK For TFV Required Fields#229

Merged
ckoegel merged 7 commits intomainfrom
SWI-9652
Feb 20, 2026
Merged

SWI-9652 Update SDK For TFV Required Fields#229
ckoegel merged 7 commits intomainfrom
SWI-9652

Conversation

@ckoegel
Copy link
Contributor

@ckoegel ckoegel commented Feb 20, 2026

No description provided.

@bwappsec
Copy link
Contributor

bwappsec commented Feb 20, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@ckoegel ckoegel marked this pull request as ready for review February 20, 2026 19:52
@ckoegel ckoegel requested review from a team as code owners February 20, 2026 19:52
api/openapi.yaml Outdated
example: 12-3456789
maxLength: 500
nullable: true
nullable: false

Choose a reason for hiding this comment

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

Aren't sole props exempt from supplying a BRN?

Choose a reason for hiding this comment

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

Looking for more info from our Ops team on industry requirements, but as far as API requirements go, we currently require BRN regardless of entity type

Choose a reason for hiding this comment

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

What value will SDK expect for BRN for sole props?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

BRN should no longer be required by the SDK, so sole props will work once the API backend is up to date

michaela-band
michaela-band previously approved these changes Feb 20, 2026
michaela-band
michaela-band previously approved these changes Feb 20, 2026
@ckoegel ckoegel merged commit fe926d1 into main Feb 20, 2026
28 checks passed
@ckoegel ckoegel deleted the SWI-9652 branch February 20, 2026 22:02
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.

4 participants