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

Conversion of Middleware Provider form to Angular #11673

Merged
merged 12 commits into from Oct 4, 2016

Conversation

AparnaKarve
Copy link
Contributor

@AparnaKarve AparnaKarve commented Oct 3, 2016

Convert the Middleware Provider form to Angular and leverage the validation directives that are currently used by the other providers.
PR includes specs for ems_middleware routing, controller actions like create and update and also jasmine specs.

screen shot 2016-10-03 at 4 43 30 pm

Fixes #9789
Fixes #11296

@miq-bot
Copy link
Member

miq-bot commented Oct 4, 2016

Checked commits AparnaKarve/manageiq@8145726~...0369aff with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
10 files checked, 4 offenses detected

app/views/ems_middleware/_form.html.haml

  • ⚠️ - Line 10 - Prefer to_s over string interpolation.
  • ⚠️ - Line 26 - Use hash rockets syntax.

app/views/ems_middleware/edit.html.haml

  • ⚠️ - Line 1 - Prefer to_s over string interpolation.

app/views/ems_middleware/new.html.haml

  • ⚠️ - Line 2 - Prefer to_s over string interpolation.

@AparnaKarve
Copy link
Contributor Author

@pilhuhn Can you give this a try? Thanks.

@pilhuhn
Copy link
Contributor

pilhuhn commented Oct 4, 2016

Awesome @AparnaKarve works like a charm
@miq-bot add_label providers/hawkular, euwe/yes

@pilhuhn
Copy link
Contributor

pilhuhn commented Oct 4, 2016

@miq-bot assign @dclarizio

@dclarizio dclarizio merged commit 6d556d5 into ManageIQ:master Oct 4, 2016
@dclarizio dclarizio deleted the middleware_to_angular branch October 4, 2016 16:28
@dclarizio dclarizio added this to the Sprint 48 Ending Oct 24, 2016 milestone Oct 4, 2016
chessbyte pushed a commit that referenced this pull request Oct 5, 2016
Conversion of Middleware Provider form to Angular
(cherry picked from commit 6d556d5)
@chessbyte
Copy link
Member

Euwe Backport details:

$ git log
commit 4e5a5609c757e1f4c28163fb4de4efb2a010d28b
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Tue Oct 4 09:28:21 2016 -0700

    Merge pull request #11673 from AparnaKarve/middleware_to_angular

    Conversion of Middleware Provider form to Angular
    (cherry picked from commit 6d556d5396380816a83ffc8e9e8343ea325fbc11)

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