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

Add Filters to DO Images #1742

Closed
wants to merge 1 commit into from
Closed

Add Filters to DO Images #1742

wants to merge 1 commit into from

Conversation

ccrvlh
Copy link

@ccrvlh ccrvlh commented Aug 14, 2022

Adds a private and tag filter to DigitalOcean Images

Description

DigitalOcean accepts both tag_name and private parameters to filter images and Nodes.
This PR adds these parameters using the ex_* convention.
https://docs.digitalocean.com/reference/api/api-reference/#tag/Images
Closes #1737

Status

  • done, ready for review

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
  • Documentation
  • Tests
  • ICLA (required for bigger changes)

Others

First contribution, will keep it as a Draft for now.

"""
List all the available images.

:param ex_private: Filters only user (private) images.
Copy link
Member

Choose a reason for hiding this comment

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

Would be good to clarify that None (default value) means that no filtering is performed.

Also, is there a distinction between None and False? If there isn't, I think we should just default to False and remove Optional annotation.

Copy link
Member

@Kami Kami left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution.

I added small question comment on None vs False, besides that, LGTM.

@ccrvlh ccrvlh closed this by deleting the head repository Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Add filters to DO Image List
2 participants