Skip to content

Added examples to schema of CursorPagination - #8687

Merged
lovelydinosaur merged 2 commits into
encode:masterfrom
norbertschuler:patch-1
Oct 11, 2022
Merged

lovelydinosaur merged 2 commits into
encode:masterfrom
norbertschuler:patch-1

Conversation

@norbertschuler

Copy link
Copy Markdown
Contributor

Fix #8686

Added missing examples for CursorPagination class to disable warnings in https://github.com/tfranzel/drf-spectacular and make it consistent with other pagination classes.

Fix #8686

Added missing examples for CursorPagination class to disable warnings in https://github.com/tfranzel/drf-spectacular and make it consistent with other pagination classes.
@lovelydinosaur

Copy link
Copy Markdown
Member

Thanks for this! Yup seems good.

Looks like we've got a test case that checks the schema output, which would need fixing up...

https://github.com/encode/django-rest-framework/actions/runs/3207548339/jobs/5262424890

Let me know if you need a pointer in getting that sorted.

@norbertschuler

Copy link
Copy Markdown
Contributor Author

Looks like we've got a test case that checks the schema output, which would need fixing up...

Sorry for not having checked that in first place. I have no environment set up to run the test suite rather relying on your GitHub actions. I fixed the test now.

@lovelydinosaur
lovelydinosaur merged commit 911b207 into encode:master Oct 11, 2022
@lovelydinosaur

Copy link
Copy Markdown
Member

Great, thanks!

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.

Nasty warning using CursorPagination with drf-spectacular

2 participants