Skip to content

Remove originalTransactionId#203

Merged
shreyav merged 1 commit intomainfrom
02-17-remove_originaltransactionid
Feb 18, 2026
Merged

Remove originalTransactionId#203
shreyav merged 1 commit intomainfrom
02-17-remove_originaltransactionid

Conversation

@shreyav
Copy link
Contributor

@shreyav shreyav commented Feb 18, 2026

more leftover from removing the retry quotes api

Copy link
Contributor Author

shreyav commented Feb 18, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link

github-actions bot commented Feb 18, 2026

✱ Stainless preview builds

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

kotlin

feat(api): remove originalTransactionId from SandboxSendFundsResponse

openapi

feat(api): remove originalTransactionId field from transaction

python

fix(api): remove original_transaction_id field from SandboxSendFundsResponse

typescript

fix(types): remove originalTransactionId field from SandboxSendFundsResponse
⚠️ grid-openapi studio · code

There was a regression in your SDK.
generate ❗

⚠️ grid-python studio · code

There was a regression in your SDK.
generate ❗build ⏳lint ⏳test ⏳

⚠️ grid-typescript studio · code

There was a regression in your SDK.
generate ❗build ✅lint ✅test ✅

npm install https://pkg.stainless.com/s/grid-typescript/574282c2ecf511702336fbdebdb04fc799a1a312/dist.tar.gz
⚠️ grid-kotlin studio · code

There was a regression in your SDK.
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-02-18 17:28:25 UTC

@shreyav shreyav marked this pull request as ready for review February 18, 2026 05:19
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 18, 2026

Greptile Summary

Removes the originalTransactionId field from the OutgoingTransaction schema across the source YAML and both bundled OpenAPI specs. This is a follow-up cleanup from the removal of the retry quotes API.

  • Deleted the originalTransactionId property (type: string) from OutgoingTransaction in the source schema and both bundled files
  • No remaining references to originalTransactionId exist anywhere in the repository
  • All three files are consistent with each other after the change

Confidence Score: 5/5

  • This PR is safe to merge — it cleanly removes a leftover field with no remaining references.
  • The change is a simple, consistent removal of a single unused property across three files. No references to originalTransactionId remain in the codebase, and both bundled specs are identical and consistent with the source file.
  • No files require special attention.

Important Files Changed

Filename Overview
openapi/components/schemas/transactions/OutgoingTransaction.yaml Removes originalTransactionId property from OutgoingTransaction schema source file. Clean removal with no issues.
openapi.yaml Bundled spec regenerated with originalTransactionId removed from OutgoingTransaction. Consistent with source.
mintlify/openapi.yaml Mintlify bundled spec regenerated with originalTransactionId removed from OutgoingTransaction. Consistent with source.

Class Diagram

%%{init: {'theme': 'neutral'}}%%
classDiagram
    class Transaction {
        +string id
        +string status
        +string createdAt
        +string updatedAt
    }
    class OutgoingTransaction {
        +TransactionSource source
        +CurrencyAmount sentAmount
        +CurrencyAmount receivedAmount
        +number exchangeRate
        +int64 fees
        +string quoteId
        +PaymentInstructions[] paymentInstructions
        +Refund refund
        +OutgoingRateDetails rateDetails
        +OutgoingTransactionFailureReason failureReason
    }
    Transaction <|-- OutgoingTransaction : allOf

    note for OutgoingTransaction "originalTransactionId removed\n(was: string, retry quote reference)"
Loading

Last reviewed commit: df53566

@shreyav shreyav merged commit 63b28d6 into main Feb 18, 2026
9 checks passed
@shreyav shreyav deleted the 02-17-remove_originaltransactionid branch February 18, 2026 17:18
AaryamanBhute pushed a commit that referenced this pull request Feb 18, 2026
more leftover from removing the retry quotes api
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

Comments