Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(booking): change error code on booking limits #179

Merged
merged 7 commits into from
Apr 27, 2022

Conversation

chillfox
Copy link
Contributor

Description of the change

Changes the error code for for reaching the booking limit to 410.

Benefits

Differentiates between a booking conflict and a booking limit error.

@chillfox chillfox self-assigned this Apr 21, 2022
@chillfox chillfox linked an issue Apr 21, 2022 that may be closed by this pull request
@github-actions github-actions bot added the type: enhancement new feature or request label Apr 21, 2022
@chillfox
Copy link
Contributor Author

@stakach should we return a error object like this for booking limits and clashing bookings?

{
  "error": "Booking limit reached",
  "limit": 3,
  "bookings": [
    {"BOOKING"}
  ]
}

@w-le
Copy link
Contributor

w-le commented Apr 21, 2022

I think that would be best practice, and really helpful for all API clients. I think it would be good across the whole API. Not urgent.

@chillfox chillfox marked this pull request as ready for review April 21, 2022 06:08
@chillfox chillfox requested review from stakach and removed request for stakach April 21, 2022 06:08
Copy link
Member

@stakach stakach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stakach stakach merged commit 2331207 into master Apr 27, 2022
@stakach stakach deleted the feat/booking_limits_error_code branch April 27, 2022 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement new feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change the error code for booking limits
3 participants