Skip to content

Preview for page metadata - #806

Open
manisha1997 wants to merge 2 commits into
mainfrom
preview-for-page-metadata
Open

Preview for page metadata#806
manisha1997 wants to merge 2 commits into
mainfrom
preview-for-page-metadata

Conversation

@manisha1997

Copy link
Copy Markdown
Contributor

No description provided.

manisha1997 added a commit to twilio/twilio-oai-generator that referenced this pull request Sep 2, 2026
…833)

Ruby templates always emitted the pagination-metadata scaffolding
(list_with_metadata method and the {{apiName}}PageMetadata /
{{apiName}}ListResponse classes) for every read operation, even when the
List response has no `meta` key and therefore does not support
pagination (e.g. ListProfileIdentifiers in Memory v1).

Node and Python already gate this on the x-supports-pagination
(per-operation) and hasOperationWithPagination (per-resource) extensions
computed in ApiResourceBuilder. Apply the same gating in the Ruby
templates:
- list.mustache: wrap list_with_metadata in x-supports-pagination
- api.mustache / parentContext.mustache: wrap the pageMetadata partial
in hasOperationWithPagination

Regenerated Ruby examples: this removes orphaned PageMetadata/
ListResponse classes that no list_with_metadata referenced.

Preview PR - twilio/twilio-ruby#806

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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