Skip to content

Conversation

@eccles
Copy link
Contributor

@eccles eccles commented Jul 7, 2022

Problem:
The backend does not support counting of matching assets or
access_policies in the access_policy endpoint.

Solution:
Removed count_matching_assets() and count_matching_access_polices()
methods in access_policies endpoint.

Signed-off-by: Paul Hewlett phewlett76@gmail.com

Problem:
The backend does not support counting of matching assets or
access_policies in the access_policy endpoint.

Solution:
Removed count_matching_assets() and count_matching_access_polices()
methods in access_policies endpoint.
Additionally made keyword arguments in update method mandatory.

Signed-off-by: Paul Hewlett <phewlett76@gmail.com>
@eccles eccles force-pushed the dev/eccles/remove-accesspolicies-count-matching branch from aa43de1 to 691f26a Compare July 8, 2022 07:40
Copy link
Contributor

@henry739 henry739 left a comment

Choose a reason for hiding this comment

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

lgtm

@eccles eccles merged commit 6eb59cb into main Jul 13, 2022
@eccles eccles deleted the dev/eccles/remove-accesspolicies-count-matching branch July 13, 2022 14:46
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