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

Convert router interface forms #7969

Conversation

GilbertCherrie
Copy link
Member

@GilbertCherrie GilbertCherrie commented Nov 19, 2021

Fixes #7987
Converted old forms to Carbon React and combined the add and remove interface forms into a single form.

Blocked by apis for adding and removing interfaces.

Before:
Screen Shot 2021-11-22 at 3 59 16 PM
Screen Shot 2021-11-22 at 4 02 46 PM

After:
Screen Shot 2021-12-08 at 11 12 13 AM
Screen Shot 2021-12-08 at 11 12 34 AM

@miq-bot miq-bot added the wip label Nov 19, 2021
@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch 2 times, most recently from fb3a12a to 37eace4 Compare November 22, 2021 20:50
@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch 3 times, most recently from 13a0498 to 1fe11b0 Compare December 7, 2021 18:52
@GilbertCherrie GilbertCherrie changed the title [WIP] Convert router interface forms Convert router interface forms Dec 7, 2021
@GilbertCherrie
Copy link
Member Author

@miq-bot add_reviewer @kavyanekkalapu
@miq-bot add-label enhancement

@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch from 9a7dc2e to 4bc3b34 Compare December 7, 2021 18:56
@miq-bot miq-bot removed the wip label Dec 7, 2021
@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch 2 times, most recently from a1be47f to 9610457 Compare December 7, 2021 19:56
@kavyanekkalapu kavyanekkalapu self-assigned this Dec 7, 2021
@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch from 9610457 to 7fc7a65 Compare December 8, 2021 16:10
@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch from 7fc7a65 to 4feef38 Compare December 9, 2021 19:52
@GilbertCherrie GilbertCherrie force-pushed the convert_router_interface_forms branch from 4feef38 to 1c85933 Compare December 9, 2021 20:02
@miq-bot
Copy link
Member

miq-bot commented Dec 9, 2021

Checked commit GilbertCherrie@1c85933 with ruby 2.6.3, rubocop 1.13.0, haml-lint 0.35.0, and yamllint
5 files checked, 2 offenses detected

app/views/network_router/add_interface_select.html.haml

  • ⚠️ - Line 1 - Layout/TrailingEmptyLines: Final newline missing.

app/views/network_router/remove_interface_select.html.haml

  • ⚠️ - Line 1 - Layout/TrailingEmptyLines: Final newline missing.

@kavyanekkalapu kavyanekkalapu merged commit b9c0f91 into ManageIQ:master Dec 9, 2021
@GilbertCherrie GilbertCherrie deleted the convert_router_interface_forms branch December 10, 2021 18:05
@GilbertCherrie GilbertCherrie mentioned this pull request Jun 8, 2022
33 tasks
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.

Form conversion: Add and remove interface in network routers
4 participants