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

Added support for collection specific hide_resources option. #78

Merged

Conversation

abellotti
Copy link
Member

This is needed for the new automate_workspaces collection where
we don't want to return current set of workspaces as they
are only accessible by resource guid from automate.

We're leveraging the current mechanism we have in place for
the hide=resources parameter and supported at the collection
level via the api.yml.

We've contemplated the 404 on the collection but that breaks
the clients and inhibits it from getting collection level
actions and such.

This is needed for the new automate_workspaces collection where
we don't want to return current set of workspaces as they
are only accessible by resource guid from automate.

We're leveraging the current mechanism we have in place for
the hide=resources parameter and supported at the collection
level via the api.yml.

We've contemplated the 404 on the collection but that breaks
the clients and inhibits it from getting collection level
actions and such.
@miq-bot
Copy link
Member

miq-bot commented Sep 26, 2017

Checked commit abellotti@cd6a932 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🍰

@mkanoor
Copy link
Contributor

mkanoor commented Sep 27, 2017

👍

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

👍

@gtanzillo gtanzillo added this to the Sprint 70 Ending Oct 2, 2017 milestone Sep 27, 2017
@gtanzillo gtanzillo merged commit 037cf40 into ManageIQ:master Sep 27, 2017
@abellotti abellotti deleted the collection_hide_resources_option branch October 6, 2017 13:03
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.

4 participants