Skip to content

remove list quotes endpoint#321

Open
shreyav wants to merge 1 commit intomainfrom
04-07-remove_list_quotes_endpoint
Open

remove list quotes endpoint#321
shreyav wants to merge 1 commit intomainfrom
04-07-remove_list_quotes_endpoint

Conversation

@shreyav
Copy link
Copy Markdown
Contributor

@shreyav shreyav commented Apr 8, 2026

No description provided.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 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 Apr 8, 2026 3:02am

Request Review

@shreyav shreyav marked this pull request as ready for review April 8, 2026 02:48
Copy link
Copy Markdown
Contributor Author

shreyav commented Apr 8, 2026

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

✱ Stainless preview builds

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

kotlin

feat(api): remove list method from quotes

openapi

feat(api): remove list method from quotes

python

feat(api): remove list method from quotes

typescript

feat(api): remove list method from quotes

Edit this comment to update them. They will appear in their respective SDK's changelogs.

⚠️ grid-python studio · code · diff

Your SDK build had at least one "error" diagnostic, which is a regression from the base state.
generate ❗ (prev: generate ✅) → build ✅lint ❗test ✅

pip install https://pkg.stainless.com/s/grid-python/f06e4bcb28e785ad1e09d7125d71cd6f859c6d30/grid-0.0.1-py3-none-any.whl
New diagnostics (1 error)
Endpoint/NotFound: Skipped endpoint because it's not in your OpenAPI spec: `get /quotes`
⚠️ grid-typescript studio · code · diff

Your SDK build had at least one "error" diagnostic, which is a regression from the base state.
generate ❗ (prev: generate ✅) → build ✅lint ❗test ✅

npm install https://pkg.stainless.com/s/grid-typescript/30e679bc624a331f3e69e4e2a833be2bcadd88e8/dist.tar.gz
New diagnostics (1 error)
Endpoint/NotFound: Skipped endpoint because it's not in your OpenAPI spec: `get /quotes`
⚠️ grid-openapi studio · code · diff

Your SDK build had at least one "error" diagnostic, which is a regression from the base state.
generate ❗ (prev: generate ✅)

New diagnostics (1 error)
Endpoint/NotFound: Skipped endpoint because it's not in your OpenAPI spec: `get /quotes`
⚠️ grid-kotlin studio · code · diff

Your SDK build had at least one "error" diagnostic, which is a regression from the base state.
generate ❗ (prev: generate ✅) → build ✅lint ✅test ✅

New diagnostics (1 error)
Endpoint/NotFound: Skipped endpoint because it's not in your OpenAPI spec: `get /quotes`

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-04-08 03:06:37 UTC

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 8, 2026

Greptile Summary

This PR removes the GET /quotes (list transfer quotes) endpoint from the API spec and the Stainless SDK config. The removal is applied consistently across the source spec (openapi/paths/quotes/quotes.yaml), both generated bundles (openapi.yaml and mintlify/openapi.yaml), and the SDK method mapping in .stainless/stainless.yml.

Confidence Score: 5/5

Safe to merge — clean, consistent removal across all four files with no dangling references.

All four affected files are updated consistently: source spec, both generated bundles, and the SDK config. No remaining references to listQuotes anywhere in the repo. No P0/P1 findings.

No files require special attention.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
openapi/paths/quotes/quotes.yaml Removes the get: operation (listQuotes) from the source OpenAPI spec; file ends cleanly with a trailing newline
.stainless/stainless.yml Removes the list: get /quotes SDK method mapping, consistent with the endpoint removal
openapi.yaml Generated bundle updated to remove the GET /quotes operation, matches source spec
mintlify/openapi.yaml Mintlify copy of the generated bundle updated to remove GET /quotes, matches source spec

Sequence Diagram

sequenceDiagram
    participant Client
    participant API as Grid API

    Note over Client,API: Remaining /quotes endpoints
    Client->>API: POST /quotes (create)
    API-->>Client: Quote object

    Client->>API: GET /quotes/{quoteId} (retrieve)
    API-->>Client: Quote object

    Client->>API: POST /quotes/{quoteId}/execute
    API-->>Client: Executed quote

    Note over Client,API: Removed endpoint
    Client--xAPI: GET /quotes (list) removed
Loading

Reviews (2): Last reviewed commit: "remove list quotes endpoint" | Re-trigger Greptile

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.

1 participant