Skip to content

[SDK-3173] Default to 'None' for deployed on GET /api/v2/actions/actions endpoint#309

Merged
adamjmcgrath merged 2 commits intomasterfrom
sdk-3173/fix-get_actions
Mar 10, 2022
Merged

[SDK-3173] Default to 'None' for deployed on GET /api/v2/actions/actions endpoint#309
adamjmcgrath merged 2 commits intomasterfrom
sdk-3173/fix-get_actions

Conversation

@evansims
Copy link
Copy Markdown
Contributor

@evansims evansims commented Mar 9, 2022

Changes

For the GET /api/v2/actions/actions endpoint, we currently (incorrectly) default the deployed parameter to False instead of None. This means we only return results containing actions that haven't been deployed. The API supports and defaults to this being unset, so all action types are returned, as one would expect.

This PR changes the parameter default from False to None to return all actions without a deployment filter applied, as would be expected, default behavior.

References

Testing

  • Tests have been updated to capture regression potential, and to cover the new None default state of the parameter.
  • Run coverage run --m unittest discover to try these updated tests, or review CircleCI logs.

Checklist

@evansims evansims requested a review from a team as a code owner March 9, 2022 21:42
@adamjmcgrath adamjmcgrath merged commit 719bc9e into master Mar 10, 2022
@adamjmcgrath adamjmcgrath deleted the sdk-3173/fix-get_actions branch March 10, 2022 16:28
@adamjmcgrath adamjmcgrath added this to the 3.22.0 milestone Mar 22, 2022
@adamjmcgrath adamjmcgrath mentioned this pull request Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using v3.19.0 unable to return all actions

2 participants