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

Disable cloud network editing and deletion for list view. #3349

Merged
merged 1 commit into from
Jan 31, 2018

Conversation

martinpovolny
Copy link
Member

Network editing is only implemented for OpenStack. So it needs to be
disabled for the general case.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1539723

@miq-bot add_label gaprindashvili/yes
@miq-bot add_label bug
@miq-bot add_label networks

Network editing is only implemented for OpenStack. So it needs to be
disabled for the general case.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1539723
@miq-bot
Copy link
Member

miq-bot commented Jan 31, 2018

Checked commit martinpovolny@e07f1cd with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

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

@mzazrivec mzazrivec self-assigned this Jan 31, 2018
@mzazrivec mzazrivec added this to the Sprint 79 Ending Feb 12, 2018 milestone Jan 31, 2018
@mzazrivec mzazrivec merged commit a025a28 into ManageIQ:master Jan 31, 2018
simaishi pushed a commit that referenced this pull request Jan 31, 2018
Disable cloud network editing and deletion for list view.
(cherry picked from commit a025a28)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540755
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 555b25d32b963d8c5011c00a4decdfc4cbf8d6a1
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Wed Jan 31 14:38:17 2018 +0100

    Merge pull request #3349 from martinpovolny/bz_1539723_cloud_network
    
    Disable cloud network editing and deletion for list view.
    (cherry picked from commit a025a2836de87c3bdeab76fca04c2dd11e5bd48a)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540755

@Loicavenel
Copy link

@martinpovolny I need to understand better this PR here, did we completely removed it from list view? Because even if an action does not apply to all Objects, we should be able to do multi-select and just report in flash message what does not apply..

simaishi pushed a commit that referenced this pull request Apr 23, 2018
Disable cloud network editing and deletion for list view.
(cherry picked from commit a025a28)

https://bugzilla.redhat.com/show_bug.cgi?id=1544488
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 63547472e046d286bf38276560e650bc3fd17da3
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Wed Jan 31 14:38:17 2018 +0100

    Merge pull request #3349 from martinpovolny/bz_1539723_cloud_network
    
    Disable cloud network editing and deletion for list view.
    (cherry picked from commit a025a2836de87c3bdeab76fca04c2dd11e5bd48a)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1544488

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.

7 participants