Skip to content

Реализация базового класса для пагинации#249

Open
thereareyou123 wants to merge 17 commits intodevfrom
feature/pagination_base_class
Open

Реализация базового класса для пагинации#249
thereareyou123 wants to merge 17 commits intodevfrom
feature/pagination_base_class

Conversation

@thereareyou123
Copy link
Copy Markdown

Прикинул примерное решение по задаче, хотелось бы узнать что поправить, что добавить

Copy link
Copy Markdown
Contributor

@AndreyZherdetskiy AndreyZherdetskiy left a comment

Choose a reason for hiding this comment

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

Посмотри, пожалуйста
https://github.com/uriyyo/fastapi-pagination

Попробуй еще реализовать с помощью вышеуказанной библиотеки

@AndreyZherdetskiy AndreyZherdetskiy added the enhancement New feature or request label Mar 17, 2025
Copy link
Copy Markdown
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

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

Как предложение. Может стоит рассмотреть такой вот пакет?
https://github.com/uriyyo/fastapi-pagination

@thereareyou123
Copy link
Copy Markdown
Author

Я добавил fastapi-pagination и подключил пагинацию в main.py, пока эндпоинты не стал трогать, так как там с ними еще вроде как активно работают, это видимо в последнюю очередь придется делать

Copy link
Copy Markdown
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

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

Хорошо, но пробовал применять на какой нибудь ручке?

@thereareyou123
Copy link
Copy Markdown
Author

Да, все ок

@thereareyou123
Copy link
Copy Markdown
Author

Я еще отдельно пустой проект с одной страничкой создавал, проверял на работоспособность и пробовал вставить в другой свой проект, все ок было

@KonstantinRaikhert
Copy link
Copy Markdown
Member

Конфликты поправить и можно мержить

@thereareyou123
Copy link
Copy Markdown
Author

Исправил все конфликты, жду тогда апрув

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

None yet

Development

Successfully merging this pull request may close these issues.

Реализация базового класса для пагинации

3 participants