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

get_all() не работает с параметром limit() #15

Closed
3 tasks
leshchenko1979 opened this issue Jul 26, 2020 · 1 comment
Closed
3 tasks
Labels
duplicate This issue or pull request already exists enhancement New feature or request

Comments

@leshchenko1979
Copy link
Owner

leshchenko1979 commented Jul 26, 2020

Выдаёт больше лимита, что вполне объяснимо, так как алгоритм get_all() не анализирует наличие лимита в params, а только ориентируется на total в ответах сервера.

  • Можно сделать параметр отдельный параметр limit, который можно будет вызывать как при обращении к get_all(), так и в 'params'.
  • Метод должен конструировать запросы к серверу таким образом, чтобы вовремя остановиться.
  • Опасность представляет возможный возврат дублей сервером при отсутствии сортировки. в параметрах запроса
@leshchenko1979 leshchenko1979 added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Jul 26, 2020
@leshchenko1979
Copy link
Owner Author

Дубль #29

@leshchenko1979 leshchenko1979 added the duplicate This issue or pull request already exists label Jul 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant