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

Fixed 'Catalog' should not be a required field #514

Merged
merged 2 commits into from
Mar 2, 2017

Conversation

h-kataria
Copy link
Contributor

@h-kataria h-kataria commented Feb 28, 2017

#@epwinchell can you help with formatting of Create New Dialog text-box.

catalog_item_form

@h-kataria h-kataria changed the title Fixed 'Catalog' should not be a required field [WIP] - Fixed 'Catalog' should not be a required field Feb 28, 2017
@miq-bot
Copy link
Member

miq-bot commented Feb 28, 2017

Checked commits h-kataria/manageiq-ui-classic@affcd7b~...185a24d with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 👍

@h-kataria h-kataria removed the wip label Feb 28, 2017
@h-kataria h-kataria changed the title [WIP] - Fixed 'Catalog' should not be a required field Fixed 'Catalog' should not be a required field Feb 28, 2017
@h-kataria h-kataria closed this Feb 28, 2017
@h-kataria h-kataria reopened this Feb 28, 2017
@h-kataria
Copy link
Contributor Author

@mkanoor @bzwei please test.

@epwinchell
Copy link
Contributor

Updated Dialog Options

screen shot 2017-02-28 at 2 54 27 pm
screen shot 2017-02-28 at 2 54 20 pm

@mkanoor
Copy link
Contributor

mkanoor commented Mar 1, 2017

👍 looks good

@bzwei
Copy link
Contributor

bzwei commented Mar 1, 2017

Isn't the desired feature when Display in Catalog is on, the Catalog field becomes mandatory?

@h-kataria
Copy link
Contributor Author

@bzwei no Catalog field is not mandatory. If user doesn't select a Catalog, Catalog Item will end up in unassigned folder.

@dclarizio dclarizio merged commit 4cd4b34 into ManageIQ:master Mar 2, 2017
@dclarizio dclarizio added this to the Sprint 56 Ending Mar 13, 2017 milestone Mar 2, 2017
h-kataria added a commit to h-kataria/manageiq-ui-classic that referenced this pull request Mar 6, 2017
This was still causing catalog drop down to be a required field and not enabling "Add" button on screen. Catalog drop down was changed to be a not required field in ManageIQ#514
h-kataria added a commit to h-kataria/manageiq-ui-classic that referenced this pull request Mar 6, 2017
This was still causing catalog drop down to be a required field and not enabling "Add" button on screen. Catalog drop down was changed to be a not required field in ManageIQ#514
@h-kataria h-kataria deleted the catalog_item_fixes branch March 15, 2017 21:00
TomasKohoutek pushed a commit to TomasKohoutek/manageiq-ui-classic that referenced this pull request Apr 7, 2017
This was still causing catalog drop down to be a required field and not enabling "Add" button on screen. Catalog drop down was changed to be a not required field in ManageIQ#514
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.

6 participants