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

Fixed optional field for GetAvailabilityResponse #169

Merged

Conversation

mrashed-dev
Copy link
Contributor

Description

In the response object for getting availability, order is only populated for round-robin events.

License

I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Copy link
Contributor

@atejada atejada left a comment

Choose a reason for hiding this comment

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

Sorry @mrashed-dev but this fix gets me this error

Required value 'timeSlots' (JSON name 'time_slots') missing at $

I'm trying to find the root cause but I can't

@mrashed-dev
Copy link
Contributor Author

@atejada fixed!

Copy link
Contributor

@atejada atejada left a comment

Choose a reason for hiding this comment

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

@mrashed-dev Hate to be this kind of guy, but...

class com.nylas.models.GetAvailabilityResponse cannot be cast to class com.nylas.models.Response (com.nylas.models.GetAvailabilityResponse and com.nylas.models.Response are in unnamed module of loader 'app')

😭

@mrashed-dev
Copy link
Contributor Author

@atejada not at all, sorry for the bugs. Thanks for testing, I think we should be good now but please test and let me know!

@mrashed-dev mrashed-dev requested a review from atejada November 14, 2023 23:29
Copy link
Contributor

@atejada atejada left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks @mrashed-dev now it works 🥳

@mrashed-dev mrashed-dev merged commit 40c37c9 into v2.0.0-beta Nov 15, 2023
1 check passed
@mrashed-dev mrashed-dev deleted the TW-2425-get-availability-error-in-kotlin-java branch November 15, 2023 16:14
mrashed-dev added a commit that referenced this pull request Nov 21, 2023
# Changelog

### Added
* Added support for the free-busy endpoint (#163)
* Added support for Messages, Drafts, and Smart Compose APIs (#166)
* Added support for custom authentication, connectors, and credentials
APIs (#165)
* Added support for folders API (#168)
* Added support for attachments API (#171)
* Added getter for `GetFreeBusyResponse` `object` field (#172)

### Changed
* Return webhook object with secret when creating a webhook (#162)
* Fixed HTTP method for rotating webhook secret (#164)
* Fixed Redirect URI endpoint path (#167)
* Fixed query parameter serialization (#167)
* Fixed optional field for `GetAvailabilityResponse` (#169)
* Fixed optional field for `Conferencing` (#170)

# License
<!-- Your PR comment must contain the following line for us to merge the
PR. -->
I confirm that this contribution is made under the terms of the MIT
license and that I have the authority necessary to make this
contribution on behalf of its copyright owner.
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.

2 participants