Skip to content

Conversation

@mrashed-dev
Copy link
Contributor

Description

A recent API update caused updating an event using the SDK to return an error because the SDK sends a Participant's status. This PR ensures that the Participant object omits it on outgoing API calls.

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.

@mrashed-dev mrashed-dev requested a review from chitreshd December 7, 2021 20:49
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #75380: 400 Error returned when updating Event with participants.

return this;
}

Map<String, Object> getWritableFields() {
Copy link

Choose a reason for hiding this comment

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

could use HashMap<String, String> here

Copy link

@jseller jseller left a comment

Choose a reason for hiding this comment

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

Looks good, could just use hashmap with strings, but if you aren't using it as a comparable it doesn't really matter

@mrashed-dev mrashed-dev merged commit 800d35b into main Dec 8, 2021
@mrashed-dev mrashed-dev deleted the mostafarashed/sc-75380/400-error-returned-when-updating-event-with branch December 8, 2021 18:05
@mrashed-dev mrashed-dev mentioned this pull request Dec 8, 2021
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.

3 participants