Skip to content

Conversation

@josecelano
Copy link
Member

@josecelano josecelano commented May 8, 2023

Changes in GET torrents response pagination.

The minimum page size for GET torrents results was removed. You can request only one torrent.

BREAKING CHANGE: a maximum page size was added (30). If you request more than 30 torrents per page, the result will contain 30 torrents at the most.

  • Remove the minimum page size
  • Add a maximum page size
  • Add configuration options in config.toml for the default and maximum page size.

@josecelano josecelano linked an issue May 8, 2023 that may be closed by this pull request
josecelano added 2 commits May 8, 2023 20:22
…page size

Changes in GET torrents response pagination.

Minimum page size for GET torrents results was removed.

A default GET torrents page size was added (10).

BREAKING CHANGE: a maximum page size was added (30). If you request more
than 30 torrents per page, the result will contain 30 torrents at the most.
```toml
[api]
default_torrent_page_size = 10
max_torrent_page_size = 30
```

With pagination options for torrents.
@josecelano josecelano marked this pull request as ready for review May 8, 2023 21:10
@josecelano josecelano requested review from da2ce7 and mickvandijke May 8, 2023 21:10
@josecelano
Copy link
Member Author

ACK 4764825

@josecelano josecelano merged commit f90c1ff into torrust:develop May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Bug: /torrents?page_size parameter is not working as expected.

1 participant