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

Validate section values for adding new custom attributes via the API #240

Merged
merged 1 commit into from
Dec 11, 2017

Conversation

jntullo
Copy link

@jntullo jntullo commented Dec 8, 2017

Custom Attributes have restrictions for the sections that are allowed to be added via the API. Currently validation only occurs for providers, but should be consistent throughout for other types. This change essentially updates the code to remove it from providers and make it accessible to all new custom attribute types.

@miq-bot add_label bug, blocker, gaprindsahvili/yes

https://bugzilla.redhat.com/show_bug.cgi?id=1516762

Currently validation only occurs for providers, but should be consistent throughout.

https://bugzilla.redhat.com/show_bug.cgi?id=1516762
@miq-bot
Copy link
Member

miq-bot commented Dec 8, 2017

@jntullo Cannot apply the following label because they are not recognized: gaprindsahvili/yes

@miq-bot
Copy link
Member

miq-bot commented Dec 8, 2017

Checked commit jntullo@47010ce with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 0 offenses detected
Everything looks fine. 👍

@imtayadeway imtayadeway self-assigned this Dec 11, 2017
@imtayadeway imtayadeway added this to the Sprint 75 Ending Dec 11, 2017 milestone Dec 11, 2017
@imtayadeway imtayadeway merged commit 9563b42 into ManageIQ:master Dec 11, 2017
simaishi pushed a commit that referenced this pull request Dec 12, 2017
Validate section values for adding new custom attributes via the API
(cherry picked from commit 9563b42)

https://bugzilla.redhat.com/show_bug.cgi?id=1525047
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit e1bb601c763e395d8d3f534404ec374601bf3d20
Author: Tim Wade <imtayadeway@users.noreply.github.com>
Date:   Mon Dec 11 10:14:05 2017 -0800

    Merge pull request #240 from jntullo/bz_1516762
    
    Validate section values for adding new custom attributes via the API
    (cherry picked from commit 9563b42a08180444df4c7be4536d9c732fd63ecc)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1525047

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