Skip to content
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.

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

fix(python): multipart file upload parameter fixes, python core utility bugfix. #5201

Merged
merged 17 commits into from
Nov 19, 2024

Conversation

eyw520
Copy link
Contributor

@eyw520 eyw520 commented Nov 18, 2024

This PR implements two fixes:

  1. Fixed issue in generated multifile upload endpoints where OMIT sentinel values could be unintentionally encoded, throwing a TypeError exception.
  2. Fixed bug in shared file.py method with_content_type() where we resolved to default spec content-type rather than user-specified content-type value.

Copy link

github-actions bot commented Nov 18, 2024

@eyw520 eyw520 merged commit e056fb4 into main Nov 19, 2024
58 of 60 checks passed
@eyw520 eyw520 deleted the eden/file-upload-parameter-fixes branch November 19, 2024 16:48
fern-bot pushed a commit that referenced this pull request Nov 19, 2024
…ty bugfix. (#5201)

* fix(python): omitted body parameters should not be encoded during files construction.

* fix(python): with_content_type should resolve to specified file content type when provided.

* Update `test_file.py` and improve `with_content_type` definition.

* Add docstring to with_content_type.

* Update python sdk seed snapshots.

* Minor fix; update `file-upload` seed test snapshot to include new changes.

* Snapshot updates.

* Snapshot updates.

* Update snapshots.

* Update ts and java snapshots.

* update
eyw520 added a commit that referenced this pull request Nov 19, 2024
* Remove `fetch-mock-jest` dependency.

* chore(docs): add example of landing page (#5205)

* add landing-page schema

* add example of landing page

* chore(docs): update example to VapiAI

* fix(docs): use accesible linking practices

---------

Co-authored-by: Danny Sheridan <83524670+dannysheridan@users.noreply.github.com>
Co-authored-by: Deep Singhvi <deep@buildwithfern.com>

* chore(docs): hidden, overview section (#5214)

hidden, overview section

Co-authored-by: fern-bot <info@buildwithfern.com>

* chore(docs): initial restructuring (#5196)

* initial restructuring

* chore: update changelog

---------

Co-authored-by: fern-bot <info@buildwithfern.com>
Co-authored-by: chdeskur <chdeskur@users.noreply.github.com>

* fix(python): multipart file upload parameter fixes, python core utility bugfix. (#5201)

* fix(python): omitted body parameters should not be encoded during files construction.

* fix(python): with_content_type should resolve to specified file content type when provided.

* Update `test_file.py` and improve `with_content_type` definition.

* Add docstring to with_content_type.

* Update python sdk seed snapshots.

* Minor fix; update `file-upload` seed test snapshot to include new changes.

* Snapshot updates.

* Snapshot updates.

* Update snapshots.

* Update ts and java snapshots.

* update

* Update seed snapshots.

---------

Co-authored-by: Elissa Alarmani <91026581+elissa-alarmani@users.noreply.github.com>
Co-authored-by: Danny Sheridan <83524670+dannysheridan@users.noreply.github.com>
Co-authored-by: Deep Singhvi <deep@buildwithfern.com>
Co-authored-by: Catherine Deskur <46695336+chdeskur@users.noreply.github.com>
Co-authored-by: fern-bot <info@buildwithfern.com>
Co-authored-by: chdeskur <chdeskur@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants