Skip to content

refactor: adapt imports to FastCRUD 0.19.0 pagination structure.#215

Merged
igorbenav merged 4 commits intobenavlabs:mainfrom
carlosplanchon:main
Nov 10, 2025
Merged

refactor: adapt imports to FastCRUD 0.19.0 pagination structure.#215
igorbenav merged 4 commits intobenavlabs:mainfrom
carlosplanchon:main

Conversation

@carlosplanchon
Copy link
Copy Markdown
Contributor

  • Replaced all occurrences of:

    from fastcrud.pagination import ...

    with:

    from fastcrud import ...
  • Ensured compatibility with FastCRUD >= 0.19.0
  • No functional or behavioral changes introduced.

You may also want to update the FastCRUD version in pyproject.toml to reflect this requirement.

Comment thread docs/user-guide/database/crud.md Outdated

```python
from fastcrud.paginated import compute_offset, paginated_response
from fastcrud import compute_offset
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can just import both in the same line

Copy link
Copy Markdown
Collaborator

@igorbenav igorbenav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just change the imports, I'll merge and deploy once the code is also updated to support fastcrud 0.19.0

Comment thread docs/user-guide/development.md Outdated

from fastapi import APIRouter, Depends, HTTPException, Request
from fastcrud.paginated import PaginatedListResponse, compute_offset
from fastcrud import PaginatedListResponse
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here as well

Copy link
Copy Markdown
Collaborator

@igorbenav igorbenav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working

@igorbenav igorbenav merged commit ad86b28 into benavlabs:main Nov 10, 2025
3 checks passed
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