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

Enhance NetworkRouter model for Amazon #16030

Merged

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented Sep 25, 2017

Enhance NetworkRouter model for Amazon

Partially implements:
https://bugzilla.redhat.com/show_bug.cgi?id=1488135

Enhance NetworkRouter model for Amazon

Partially implements:
https://bugzilla.redhat.com/show_bug.cgi?id=1488135
@Ladas
Copy link
Contributor Author

Ladas commented Sep 25, 2017

@miq-bot assign @agrare
@miq-bot add_label enhancement

cc @bronaghs AWS router RFE

@miq-bot
Copy link
Member

miq-bot commented Sep 25, 2017

Checked commit Ladas@0360a88 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 2 offenses detected

app/models/network_router.rb

Copy link
Member

@agrare agrare left a comment

Choose a reason for hiding this comment

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

👍

@agrare agrare merged commit 7ce93a9 into ManageIQ:master Sep 26, 2017
@agrare agrare added this to the Sprint 70 Ending Oct 2, 2017 milestone Sep 26, 2017
@simaishi
Copy link
Contributor

@Ladas ManageIQ/manageiq-ui-classic#2234 is now marked as fine/yes. Can this PR be fine/yes?

@simaishi
Copy link
Contributor

@Ladas ^ ping

@Ladas
Copy link
Contributor Author

Ladas commented Apr 19, 2018

@simaishi I think we should mark the UI PR ManageIQ/manageiq-ui-classic#2234 as fine/no. The other backend PRs are Fine/no, I think the backend would have to be reworked for the Fine

@simaishi
Copy link
Contributor

@Ladas thanks!

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

4 participants