Conversation
There was a problem hiding this comment.
Potentially add pagination and sorting like in /applications (not done)
|
Also, if its easier for you, you can now also branch inside this repo without needing your fork if thats easier for you. |
|
Ok the more i think about it, the more i agree that we shouldnt duplicate routes the way i intended. |
Ah yeah good point! I'll do that for my next PRs
Yes as long as you don't want to list claims for every team. I'm not super familiar with how they're used so maybe it always makes sense to list claims for one team at a time. I'll make that change for this route and add pagination. |
|
Pushed a commit to make auth optional for the route, defaulting It would be nice to make I'll add pagination when I get a chance later this week. |
|
I think what you did is good (that also how the docs say you should handle it). Not sure about type saftey though. |
This PR implements
GET /claims, with some other minor changes.Want to confirm how auth should work for this route, it looks like v1 didn't check auth for this route, although v2 requires build team token auth by default.