Skip to content

Feat: upgrade strawberry-graphql-django 0.57 -> 0.86 and set PAGINATION MAX LIMIT#265

Open
susilnem wants to merge 1 commit into
developfrom
feature/update-django-strawberry
Open

Feat: upgrade strawberry-graphql-django 0.57 -> 0.86 and set PAGINATION MAX LIMIT#265
susilnem wants to merge 1 commit into
developfrom
feature/update-django-strawberry

Conversation

@susilnem

Copy link
Copy Markdown
Member

Changes

  • Bump strawberry-graphql-django to ~=0.86.0 (core auto-updated to 0.319.0)
  • Add PAGINATION_MAX_LIMIT=100 to STRAWBERRY_DJANGO settings
  • Add pagination limit tests in apps/common/tests/pagination_test.py

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests
  • permission checks (tests here too)
  • translations

@susilnem susilnem force-pushed the feature/update-django-strawberry branch from 7b591bd to cea790b Compare June 23, 2026 09:17
- Bump strawberry-graphql-django to ~=0.86.0 (core auto-updated to 0.319.0)
- Add PAGINATION_MAX_LIMIT=100 to STRAWBERRY_DJANGO settings
- Add pagination limit tests in apps/common/tests/pagination_test.py
@susilnem susilnem force-pushed the feature/update-django-strawberry branch from cea790b to 8dfe04b Compare June 23, 2026 11:09
@codecov

codecov Bot commented Jun 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.60%. Comparing base (189901a) to head (8dfe04b).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #265      +/-   ##
===========================================
+ Coverage    87.55%   87.60%   +0.04%     
===========================================
  Files          211      212       +1     
  Lines        12819    12860      +41     
  Branches      1089     1089              
===========================================
+ Hits         11224    11266      +42     
+ Misses        1196     1195       -1     
  Partials       399      399              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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.

2 participants