Skip to content

Prepare 0.26 #446

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

Merged
merged 6 commits into from
May 26, 2020
Merged

Prepare 0.26 #446

merged 6 commits into from
May 26, 2020

Conversation

phadej
Copy link
Contributor

@phadej phadej commented May 26, 2020

No description provided.

phadej and others added 6 commits May 26, 2020 22:05
I have encountered situations where the `repoWebhookResponseStatus` field returned by GitHub is `null` (in my case it was in last_response).  When this happens, I get the following error:

```
ParseError "Error in $[1]['last_response'].status: expected Text, encountered Null"
```

Unfortunately I can't find anything in the documentation (https://developer.github.com/v3/repos/hooks/) indicating that the status field is nullable.  At any rate, this PR fixes the problem for me.
Add privacy field to types for teams

The privacy field was previously commented out while the privacy level
of a team was part of the `ironman` API preview. This feature is now
part of the official v3 API, so this commit adds support for this field.

Make privacy field non-optional in teams

The `privacy` field is optional when creating/editing teams, but
has a context dependent default value. When getting the team details,
the field is expected to always be populated due to these default
values.

Update team privacy/permission optionality

Makes team `privacy` and `permission` fields required when creating a
team so users are forced to choose, but optional when editing a team.

Filter null values from types for teams
@phadej phadej merged commit f1c34a1 into master May 26, 2020
@phadej phadej deleted the prepare-0.26 branch May 26, 2020 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants