Skip to content

Revert "[kyb] add issuing authority to IDs"#482

Merged
gcamata merged 1 commit into
mainfrom
revert-480-05-19-_kyb_add_issuing_authority_to_ids
May 20, 2026
Merged

Revert "[kyb] add issuing authority to IDs"#482
gcamata merged 1 commit into
mainfrom
revert-480-05-19-_kyb_add_issuing_authority_to_ids

Conversation

@gcamata
Copy link
Copy Markdown
Contributor

@gcamata gcamata commented May 20, 2026

Reverts #480

oops. realized we want this on the document object, not on the beneficial owner

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
grid-flow-builder Ready Ready Preview, Comment May 20, 2026 12:14am

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 20, 2026

✱ Stainless preview builds for grid

This PR will update the grid SDKs with the following commit messages.

cli

fix(api): remove issuing-authority from beneficial owners create/update

csharp

feat(api): remove IssuingAuthority field from beneficial owners

go

feat(api): remove IssuingAuthority field from beneficial owner personal info

kotlin

feat(api): remove issuingAuthority field from beneficial owners personal info

openapi

feat(api): remove issuingAuthority field from beneficial owner identification models

php

fix(api): remove issuingAuthority field from beneficial owners

python

fix(api): remove issuing_authority field from beneficial_owners personal info

ruby

fix(api): remove issuing_authority field from beneficial owner models

typescript

fix(types): remove issuingAuthority from beneficial owners
⚠️ grid-openapi studio · code

Your SDK build had at least one warning diagnostic.
generate ✅

⚠️ grid-ruby studio · code

Your SDK build had a failure in the lint CI job, which is a regression from the base state.
generate ⚠️build ✅lint ❗test ✅

⚠️ grid-go studio · code

Your SDK build had a failure in the lint CI job, which is a regression from the base state.
generate ❗build ✅lint ❗test ❗

go get github.com/stainless-sdks/grid-go@7b293cf6b68c32c5a0ab40fc4c17fe96116398f8
⚠️ grid-kotlin studio · code

Your SDK build had a failure in the test CI job, which is a regression from the base state.
generate ✅build ✅lint ✅test ❗

⚠️ grid-python studio · code

Your SDK build had at least one warning diagnostic.
generate ✅build ✅lint ✅test ✅

pip install https://pkg.stainless.com/s/grid-python/a575e2ec9c63b50327647ed860ea1d7fd566ef64/grid-0.0.1-py3-none-any.whl
⚠️ grid-typescript studio · code

Your SDK build had a failure in the build CI job, which is a regression from the base state.
generate ✅build ❗lint ❗test ❗

⚠️ grid-csharp studio · code

Your SDK build had a failure in the build CI job, which is a regression from the base state.
generate ❗build ❗lint ❗test ❗

⚠️ grid-php studio · code

Your SDK build had at least one "error" diagnostic.
generate ❗lint ✅test ✅

⚠️ grid-cli studio · code

Your SDK build had a failure in the build CI job, which is a regression from the base state.
generate ⚠️build ❗lint ❗test ❗


This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push.
If you push custom code to the preview branch, re-run this workflow to update the comment.
Last updated: 2026-05-20 00:28:25 UTC

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 20, 2026

Greptile Summary

This PR reverts #480, removing the issuingAuthority field from beneficial-owner identification schemas. The field will instead be added to the document object in a follow-up PR.

  • issuingAuthority is removed from BeneficialOwnerPersonalInfo, BeneficialOwnerPersonalInfoUpdate, and PersonalIdentification source schemas, with the bundled openapi.yaml and mintlify/openapi.yaml updated consistently at all affected locations.
  • All five changed files are in sync — no stale copies of the removed field remain.

Confidence Score: 5/5

Straightforward revert that removes a single optional field from three source schemas and their two bundled outputs — all copies are consistent and nothing else is touched.

The change is a clean, mechanical revert: the same four-line block is removed from each schema file, and both bundled specs (openapi.yaml and mintlify/openapi.yaml) reflect the removal at every inline occurrence. No logic, validation, or required fields are affected.

No files require special attention.

Important Files Changed

Filename Overview
openapi/components/schemas/customers/BeneficialOwnerPersonalInfo.yaml Removes issuingAuthority field from BeneficialOwnerPersonalInfo schema — clean revert of #480.
openapi/components/schemas/customers/BeneficialOwnerPersonalInfoUpdate.yaml Removes issuingAuthority field from BeneficialOwnerPersonalInfoUpdate schema — consistent with the revert.
openapi/components/schemas/customers/PersonalIdentification.yaml Removes issuingAuthority field from PersonalIdentification schema — consistent with the revert.
openapi.yaml Bundled OpenAPI spec updated at both inline schema locations to remove issuingAuthority — in sync with source schema changes.
mintlify/openapi.yaml Mintlify bundled OpenAPI spec updated at both inline schema locations to remove issuingAuthority — mirrors openapi.yaml changes.

Class Diagram

%%{init: {'theme': 'neutral'}}%%
classDiagram
    class BeneficialOwnerPersonalInfo {
        string idNumber
        string idType
        string issuingCountry
        ~~string issuingAuthority~~ removed
    }
    class BeneficialOwnerPersonalInfoUpdate {
        string idNumber
        string idType
        string issuingCountry
        ~~string issuingAuthority~~ removed
    }
    class PersonalIdentification {
        string idNumber
        string idType
        string issuingCountry
        ~~string issuingAuthority~~ removed
    }
    BeneficialOwnerPersonalInfo ..> PersonalIdentification : references
    BeneficialOwnerPersonalInfoUpdate ..> PersonalIdentification : references
Loading

Reviews (1): Last reviewed commit: "Revert "[kyb] add issuing authority to I..." | Re-trigger Greptile

@gcamata gcamata merged commit 4843f8d into main May 20, 2026
8 checks passed
@gcamata gcamata deleted the revert-480-05-19-_kyb_add_issuing_authority_to_ids branch May 20, 2026 00:22
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.

2 participants