feature/balance #13

Merged
ferfissimo merged 4 commits from feature/balance into develop 2021-08-09 10:06:54 +00:00
Owner
No description provided.
crimsen added 4 commits 2021-06-29 11:54:19 +00:00
ferfissimo reviewed 2021-07-05 14:14:17 +00:00
@ -99,6 +99,32 @@ def set_limit(userid, current_session: Session):
return HTTP.no_content()
@BalancePlugin.blueprint.route("/users/balance/limit", methods=["GET", "PUT"])
Owner

Not sure how I think about that API.
Not a friend of multiple API endpoints for quite the same task (changing the limit of an user).
But on the other hand the current enpoint requires you to send N requests (for N users)...

Not sure how I think about that API. Not a friend of multiple API endpoints for quite the same task (changing the limit of an user). But on the other hand the current enpoint requires you to send N requests (for N users)...
Owner

Still WIP or mergeable?

Still WIP or mergeable?
ferfissimo merged commit 4e46ea1ca3 into develop 2021-08-09 10:06:54 +00:00
Sign in to join this conversation.
No description provided.