-
Notifications
You must be signed in to change notification settings - Fork 0
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
Document that current user's bookings are returned if zones AND user are not specified #140
Labels
priority: low
type: documentation
improvements or additions to documentation
type: enhancement
new feature or request
Comments
w-le
changed the title
query bookings: return 400 is zones AND user is not specified
query bookings: return 400 if zones AND user is not specified
Jan 11, 2022
w-le
changed the title
query bookings: return 400 if zones AND user is not specified
query bookings: return 400 if zones AND user are not specified
Jan 11, 2022
That is intuitive behaviour actually. So yeah if it's confirmed that that is working as expected then no code change is required, we'll just update the apiary and any other documentation to let end users know |
@w-le would you prefer if I open a tracking issue for any relevant documentation changes, or just reopen this one? |
Let's just reuse this one and update the title and description
…On Tue, 18 Jan 2022, 10:03 am Caspian Baska, ***@***.***> wrote:
@w-le <https://github.com/w-le> would you prefer if I open a tracking
issue for any relevant documentation changes, or just reopen this one?
—
Reply to this email directly, view it on GitHub
<#140 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACPJ7KBXKO5KKTY5U6O7PWDUWTDAVANCNFSM5LVKCDSA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
caspiano
added
the
type: documentation
improvements or additions to documentation
label
Jan 18, 2022
caspiano
changed the title
query bookings: return 400 if zones AND user are not specified
Document that current user's bookings are returned if zones AND user are not specified
Jan 18, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
priority: low
type: documentation
improvements or additions to documentation
type: enhancement
new feature or request
https://placeoscalendar.docs.apiary.io/#reference/0/bookings/query-bookingsEither zones OR user MUST be specified. Otherwise there is no search criteria. The system will NOT search ALL bookings across all zones when
zones
is not suppliedCurrently when the request is made when
zones
anduser
are not specified, the result will ALWAYS be[]
because nothing is being searched. This is not obvious to the end user though.A better response would be something like "400: Either zones or user param MUST be specified"
Currently, when the request is made when
zones
anduser
are not specified, the result will return the current user's bookings.This behaviour should be documented in the API reference.
The text was updated successfully, but these errors were encountered: