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

Allow guests as payload and receive it in some responses #919

Merged
merged 2 commits into from
May 31, 2024

Conversation

jekku
Copy link
Contributor

@jekku jekku commented May 31, 2024

What's here?

This allows some alternative occupancy parameters for future use.

@jekku jekku requested a review from a team as a code owner May 31, 2024 09:48
src/Stays/StaysTypes.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@ulissesalmeida ulissesalmeida left a comment

Choose a reason for hiding this comment

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

LGTM, but I'll let the JS experts t review it

@jekku jekku requested a review from andrejak May 31, 2024 11:29
Copy link
Contributor

@igorp1 igorp1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@jekku jekku merged commit 91c9b5b into main May 31, 2024
4 checks passed
@jekku jekku deleted the jo-alternative-search-params branch May 31, 2024 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants