stadust / pointercrate

Rewrite of the https://pointercrate.com backend in Rust
MIT License
78 stars 50 forks source link

Do not work `position` field #48

Closed slo0ey closed 3 years ago

slo0ey commented 3 years ago

I tried filtering using a 'position' field (in many situations), but it doesn't work at all.

api/v1/demons/?position=10 Normal - returns no.10 demon Output - returns no.1\~no.50 demonlist


api/v1/demons/?limit=10&position__gt=10 Normal - returns no.11\~no.20 demonlist Output - returns no.1\~no.10 demonlist

To put the proper analogy, it's the same effect as none.

stadust commented 3 years ago

Documentation error, that field is not actually supported on that endpoint.

use /api/v2/demons/ for pagination by id and /api/v2/demons/listed/ for pagination of placed demons by position

slo0ey commented 3 years ago

Oh, I see! By the way, when are you planning to support for the position field

stadust commented 3 years ago

Not at all. See https://pointercrate.com/documentation/index/#pagination, https://pointercrate.com/documentation/demons/#demon-listing and https://pointercrate.com/documentation/demons/#demon-retrieval