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

[FINE] Corrects model and function names in New Subnet form #3254

Merged
merged 1 commit into from Jan 16, 2018
Merged

[FINE] Corrects model and function names in New Subnet form #3254

merged 1 commit into from Jan 16, 2018

Conversation

petrblaho
Copy link

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

Steps for Testing/QA [taken from BZ linked above]

Steps to Reproduce:

  1. Add OpenStack provider.
  2. Navigate to Network -> Subnets
  3. Open add new subnet page
  4. Check page UI

Results without patch:
Page is corrupted, network dropdown is missing

Results with patch (expected):
Network drop-down should be present on the page

@aufi
Copy link
Member

aufi commented Jan 16, 2018

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

@miq-bot
Copy link
Member

miq-bot commented Jan 16, 2018

Checked commit https://github.com/petrblaho/manageiq-ui-classic/commit/6cbed30f3e11a82c02c41297543e640b5a5c25d0 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@petrblaho
Copy link
Author

The code for newer branches is already there - https://github.com/ManageIQ/manageiq-ui-classic/pull/1545/files#diff-338ecb79799bff25e172f8849a48dbbd

@petrblaho
Copy link
Author

@himdel, can you or anyone can look at this, please?

@himdel himdel removed the fine/yes label Jan 16, 2018
@himdel himdel changed the title [fine] Corrects model and function names in New Subnet form [FINE] Corrects model and function names in New Subnet form Jan 16, 2018
@himdel
Copy link
Contributor

himdel commented Jan 16, 2018

LGTM, fixes #2301 (comment), thanks! :)

Copy link
Contributor

@himdel himdel left a comment

Choose a reason for hiding this comment

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

👍

@simaishi simaishi merged commit 34b8794 into ManageIQ:fine Jan 16, 2018
@simaishi simaishi added this to the Sprint 78 Ending Jan 29, 2018 milestone Jan 16, 2018
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.

None yet

5 participants