Skip to content

chore: render python params correctly #689

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 14, 2025
Merged

chore: render python params correctly #689

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1fefef0
chore: render python params correctly
tiwarishubham635 Oct 13, 2025
7d57880
chore: update examples
tiwarishubham635 Oct 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions examples/python/twilio/rest/flex_api/v1/credential/aws/history.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,16 +125,16 @@ def fetch(
:returns: The fetched HistoryInstance
"""

data = values.of({})
params = values.of({})

data.update(serialize.prefixed_collapsible_map(add_ons_data, "AddOns"))
params.update(serialize.prefixed_collapsible_map(add_ons_data, "AddOns"))

headers = values.of({})

headers["Accept"] = "application/json"

payload = self._version.fetch(
method="GET", uri=self._uri, params=data, headers=headers
method="GET", uri=self._uri, params=params, headers=headers
)

return HistoryInstance(
Expand All @@ -154,16 +154,16 @@ async def fetch_async(
:returns: The fetched HistoryInstance
"""

data = values.of({})
params = values.of({})

data.update(serialize.prefixed_collapsible_map(add_ons_data, "AddOns"))
params.update(serialize.prefixed_collapsible_map(add_ons_data, "AddOns"))

headers = values.of({})

headers["Accept"] = "application/json"

payload = await self._version.fetch_async(
method="GET", uri=self._uri, params=data, headers=headers
method="GET", uri=self._uri, params=params, headers=headers
)

return HistoryInstance(
Expand Down
134 changes: 102 additions & 32 deletions src/main/resources/twilio-python/context.handlebars
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,15 @@ class {{apiName}}Context(InstanceContext):
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}
{{/vendorExtensions.successProduce}}{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

payload = self._version.update(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data, headers=headers)
{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}

payload = self._version.update(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data, headers=headers{{#if queryParams}}, params=params{{/if}})

return {{instanceName}}(
self._version,
Expand Down Expand Up @@ -74,9 +80,15 @@ class {{apiName}}Context(InstanceContext):
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}
{{/vendorExtensions.successProduce}}{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}

payload = await self._version.update_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data, headers=headers)
payload = await self._version.update_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data, headers=headers{{#if queryParams}}, params=params{{/if}})

return {{instanceName}}(
self._version,
Expand All @@ -92,22 +104,27 @@ class {{apiName}}Context(InstanceContext):

:returns: The fetched {{instanceName}}
"""
{{#if allParams}}
data = values.of({ {{#allParams}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/allParams}}
{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

{{#allParams}}{{#if vendorExtensions.x-prefixed-collapsible-map}}data.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/allParams}}{{/if}}
{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}

headers = values.of({})
{{#allParams}}{{#isHeaderParam}}
{{^if required}}
if not ({{paramName}} is values.unset or (isinstance({{paramName}}, str) and not {{paramName}})):{{/if}}
headers['{{{baseName}}}'] = {{paramName}}
{{/isHeaderParam}}{{/allParams}}
{{#consumes}}
headers["Content-Type"] = "{{{mediaType}}}"
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}
payload = self._version.fetch(method='{{vendorExtensions.x-http-method}}', uri=self._uri {{#if allParams}}, params=data{{/if}} , headers=headers)
payload = self._version.fetch(method='{{vendorExtensions.x-http-method}}', uri=self._uri {{#if queryParams}}, params=params{{/if}} , headers=headers)

return {{instanceName}}(
self._version,
Expand All @@ -124,22 +141,27 @@ class {{apiName}}Context(InstanceContext):

:returns: The fetched {{instanceName}}
"""
{{#if allParams}}
data = values.of({ {{#allParams}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/allParams}}
{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

{{#allParams}}{{#if vendorExtensions.x-prefixed-collapsible-map}}data.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/allParams}}{{/if}}
{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}

headers = values.of({})
{{#allParams}}{{#isHeaderParam}}
{{^if required}}
if not ({{paramName}} is values.unset or (isinstance({{paramName}}, str) and not {{paramName}})):{{/if}}
headers['{{{baseName}}}'] = {{paramName}}
{{/isHeaderParam}}{{/allParams}}
{{#consumes}}
headers["Content-Type"] = "{{{mediaType}}}"
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}
payload = await self._version.fetch_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri {{#if allParams}}, params=data{{/if}}, headers=headers)
payload = await self._version.fetch_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri {{#if queryParams}}, params=params{{/if}}, headers=headers)

return {{instanceName}}(
self._version,
Expand All @@ -155,20 +177,26 @@ class {{apiName}}Context(InstanceContext):
{{#allParams}}:param {{paramName}}: {{{description}}}
{{/allParams}}
:returns: True if delete succeeds, False otherwise
"""{{#if allParams}}
headers = values.of({{#allParams}}{'{{{baseName}}}': {{paramName}}, }{{/allParams}})
"""{{#if headerParams}}
headers = values.of({{#allParams}}{{#isHeaderParam}}{'{{{baseName}}}': {{paramName}}, }{{/isHeaderParam}}{{/allParams}})
{{/if}}

{{^if allParams}}
{{^if headerParams}}
headers = values.of({})
{{/if}}
{{#consumes}}
headers["Content-Type"] = "{{{mediaType}}}"
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}
return self._version.delete(method='{{vendorExtensions.x-http-method}}', uri=self._uri, headers=headers)
{{/vendorExtensions.successProduce}}{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}
return self._version.delete(method='{{vendorExtensions.x-http-method}}', uri=self._uri, headers=headers{{#if queryParams}}, params=params{{/if}})

async def {{vendorExtensions.x-name-lower}}_async(self{{#allParams}}, {{paramName}}: {{#if required}}{{{dataType}}}{{else}}Union[{{{dataType}}}, object]=values.unset{{/if}}{{/allParams}}) -> bool:
"""
Expand All @@ -177,19 +205,25 @@ class {{apiName}}Context(InstanceContext):
{{#allParams}}:param {{paramName}}: {{{description}}}
{{/allParams}}
:returns: True if delete succeeds, False otherwise
"""{{#if allParams}}
headers = values.of({{#allParams}}{'{{{baseName}}}': {{paramName}}, }{{/allParams}})
"""{{#if headerParams}}
headers = values.of({{#allParams}}{{#isHeaderParam}}{'{{{baseName}}}': {{paramName}}, }{{/isHeaderParam}}{{/allParams}})
{{/if}}
{{^if allParams}}
{{^if headerParams}}
headers = values.of({})
{{/if}}
{{#consumes}}
headers["Content-Type"] = "{{{mediaType}}}"
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}
return await self._version.delete_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri, headers=headers)
{{/vendorExtensions.successProduce}}{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}
return await self._version.delete_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri, headers=headers{{#if queryParams}}, params=params{{/if}})
{{/vendorExtensions.x-twilio.ignoreOperation}}{{/vendorExtensions.x-is-delete-operation}}{{#vendorExtensions.x-is-create-operation}}{{^vendorExtensions.x-twilio.ignoreOperation}}
def {{vendorExtensions.x-name-lower}}(self{{#allParams}}, {{paramName}}: {{#if required}}{{{dataType}}}{{else}}Union[{{{dataType}}}, object]=values.unset{{/if}}{{/allParams}}) -> {{instanceName}}:
"""
Expand All @@ -199,11 +233,29 @@ class {{apiName}}Context(InstanceContext):

:returns: The created {{instanceName}}
"""
data = values.of({ {{#allParams}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/allParams}}
{{#bodyParam}}data = {{paramName}}.to_dict(){{/bodyParam}}
{{^bodyParam}}data = values.of({ {{#allParams}}{{#isFormParam}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/isFormParam}}{{/allParams}}
}){{/bodyParam}}
headers = values.of({})
{{#allParams}}{{#isHeaderParam}}
{{^if required}}
if not ({{paramName}} is values.unset or (isinstance({{paramName}}, str) and not {{paramName}})):{{/if}}
headers['{{{baseName}}}'] = {{paramName}}{{/isHeaderParam}}{{/allParams}}
{{#consumes}}
headers["Content-Type"] = "{{{mediaType}}}"
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

payload = self._version.create(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data)
{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}

payload = self._version.create(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data, headers=headers{{#if queryParams}}, params=params{{/if}})

return {{instanceName}}(
self._version,
Expand All @@ -219,11 +271,29 @@ class {{apiName}}Context(InstanceContext):

:returns: The created {{instanceName}}
"""
data = values.of({ {{#allParams}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/allParams}}
{{#bodyParam}}data = {{paramName}}.to_dict(){{/bodyParam}}
{{^bodyParam}}data = values.of({ {{#allParams}}{{#isFormParam}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/isFormParam}}{{/allParams}}
}){{/bodyParam}}
headers = values.of({})
{{#allParams}}{{#isHeaderParam}}
{{^if required}}
if not ({{paramName}} is values.unset or (isinstance({{paramName}}, str) and not {{paramName}})):{{/if}}
headers['{{{baseName}}}'] = {{paramName}}{{/isHeaderParam}}{{/allParams}}
{{#consumes}}
headers["Content-Type"] = "{{{mediaType}}}"
{{/consumes}}
{{#vendorExtensions.successProduce}}
headers["Accept"] = "{{{mediaType}}}"
{{/vendorExtensions.successProduce}}{{#if queryParams}}
params = values.of({ {{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}{{else}}
'{{{baseName}}}': {{#if vendorExtensions.x-serialize}}{{vendorExtensions.x-serialize}}({{paramName}}{{#if isArray}}, lambda e: e){{else}}){{/if}}{{else}}{{paramName}}{{/if}},{{/if}}{{/isQueryParam}}{{/allParams}}
})

payload = await self._version.create_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data)
{{#allParams}}{{#isQueryParam}}{{#if vendorExtensions.x-prefixed-collapsible-map}}params.update({{vendorExtensions.x-serialize}}({{paramName}}, '{{vendorExtensions.x-prefixed-collapsible-map}}'))
{{/if}}{{/isQueryParam}}{{/allParams}}{{/if}}

payload = await self._version.create_async(method='{{vendorExtensions.x-http-method}}', uri=self._uri, data=data, headers=headers{{#if queryParams}}, params=params{{/if}})

return {{instanceName}}(
self._version,
Expand Down
Loading