Add ability to add or remove access for beta groups to a build #164
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PR
This is a slightly more involved change unfortunately, but it's necessary for these endpoints to work. Both endpoints expect the
data
parameter to be anArray
and since this has always been an object (Hash
) so far, there was no facility to pass in anArray
parameter to the request. Also, theDELETE
endpoint expects a body which is not allowed inNet::HTTP::Delete
, so a customDELETE
class had to be used.The
DELETE
endpoint was already implemented but I don't see how it could have ever worked since no request body could have been passed into it. It's working now but its alias changed to reflect the endpoint - this is technically a breaking change but as I said, I don't think this was working before.I documented the usage in the README.