Skip to content
This repository was archived by the owner on Dec 20, 2023. It is now read-only.

More klassement routes: user per event, events per class and events per user #66

Merged
merged 2 commits into from
Nov 7, 2023

Conversation

tiptenbrink
Copy link
Collaborator

This contains three new routes:

  • /admin/class/events/user/{user_id}/ is for getting all events of a user in a specific class, which can be given explicitly or just referred to as "points" or "training"
  • /admin/class/events/ returns all events in a class. Again either by explicitly giving the class id or ranking type.
  • /admin/class/events/{event_id}/ gets all users and their points for a specific event, using only the event_id.

There was also some wrong syntax in the aggregation query, which e.g. affected /admin/class/sync/ (previously /admin/classification/sync/{publish}/). This has now been fixed.

The only routes left are for modifying stuff. For modifying/adding a user to an event we already have a data function, but not yet a route.

@tiptenbrink tiptenbrink merged commit a8cebd3 into main Nov 7, 2023
@tiptenbrink tiptenbrink deleted the more-klassement-routes branch November 29, 2023 08:52
@tiptenbrink tiptenbrink mentioned this pull request Nov 7, 2023
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant