An http API is required so i can reserve the setup from my command line

PimClappers commented 3 weeks ago

Imagine just having to type "reserveCorn 20" in my linux command line to reserve te setup for 20 minutes. Yes please!

PimClappers commented 3 weeks ago

Is er al ofzo?

InzeNL commented 3 weeks ago

Reopening this.

We should make it so the API returns consistent responses. Instead of return jsonify("Could not find reservation"), 404, we should return something more consistent.

Status codes should be consistently added
Satus codes should make sense with the request that's added
Creating a reservation shouldn't go through "/", but should instead have a separate endpoint. The "/" page can still do this, so the form works, but a separate endpoint should exist
Endpoints should follow a more sensible format. Use kebab-case, and resource-based urls (e.g. "/reservations")
- A "/reservations" endpoint might be good enough for everything. Have it return the current and the queue in one object.
- That way, "GET" gets all the reservations, "PUT" adds one, "DELETE" removes one

InzeNL commented 3 weeks ago

Since the task changed a bit compared to what you described, I unassigned you @PimClappers. Feel free to reassign yourself if you still want to implement this

InzeNL commented 2 weeks ago

@DavidBergevoet would it be an idea to perform the "organising of the Python files" in this task?

DavidBergevoet commented 2 weeks ago

Yeah it would be but no other branch should be open to have no weird merges or conflicts

InzeNL commented 2 weeks ago

In regards to #14, this should also create an endpoint to obtain all available setups. Along with that, a "reservations" query should have a filter option to filter by one or more setup ids

DavidBergevoet / Setup-Reservation

An http API is required so i can reserve the setup from my command line #9