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

Updating JSON schema responses #257

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Updating JSON schema responses #257

merged 1 commit into from
Oct 19, 2023

Conversation

dantownsend
Copy link
Member

Fixing the tests, and using the new json_schema_extra argument for create_pydantic_model.

@dantownsend dantownsend added the enhancement New feature or request label Oct 19, 2023
@codecov-commenter
Copy link

codecov-commenter commented Oct 19, 2023

Codecov Report

Merging #257 (c670de0) into v1 (ccbeabf) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##               v1     #257   +/-   ##
=======================================
  Coverage   92.69%   92.69%           
=======================================
  Files          33       33           
  Lines        2039     2039           
=======================================
  Hits         1890     1890           
  Misses        149      149           
Files Coverage Δ
piccolo_api/crud/endpoints.py 95.75% <ø> (ø)

@dantownsend dantownsend merged commit dc61090 into v1 Oct 19, 2023
11 checks passed
dantownsend added a commit that referenced this pull request Oct 20, 2023
* pydantic v2 support (#245)

* pydantic_v2_support

* pin to Piccolo v1

* add comment

* remove KeyError Exception

* upgrade coverage

* fix type warnings with `nested`

* fix mypy warnings

* update black

* replacement for `outer_type`

* change assertion

---------

Co-authored-by: Daniel Townsend <[email protected]>

* update github actions

* bumped version

* fix `_get_type` - check for `NoneType` (#254)

* fix `_get_type` - check for `NoneType`

* add tests

* fix typo

* add a test for the new union syntax

* also check for `UnionType`

* bumped version

* update JSON schema (#257)

* bumped version

* Update requirements.txt

---------

Co-authored-by: sinisaos <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants