[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16389233#comment-16389233 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/1913 @clebertsuconic yup, looks better, i've merged. > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16389230#comment-16389230 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1913 > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16389193#comment-16389193 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user stanlyDoge commented on the issue: https://github.com/apache/activemq-artemis/pull/1913 @clebertsuconic Done > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16388897#comment-16388897 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1913 @michaelandrepearce is this good to merge? (the PR check failure is probably not related). @stanlyDoge if you could please rebase to have a re-run? > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16387478#comment-16387478 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user stanlyDoge commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172447340 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js --- @@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) { .when('/artemis/deleteAddress', { templateUrl: ARTEMIS.templatePath + 'deleteAddress.html' }) + .when('/artemis/deleteDivert', { --- End diff -- Well, it shows page without such button. But I agree it would be better to hide this button in "toolbar". I will take a look at this. > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16387474#comment-16387474 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172446748 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js --- @@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) { .when('/artemis/deleteAddress', { templateUrl: ARTEMIS.templatePath + 'deleteAddress.html' }) + .when('/artemis/deleteDivert', { --- End diff -- Then the consoles should hide the button, if divert is selected. > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16387471#comment-16387471 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user stanlyDoge commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172446094 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js --- @@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) { .when('/artemis/deleteAddress', { templateUrl: ARTEMIS.templatePath + 'deleteAddress.html' }) + .when('/artemis/deleteDivert', { --- End diff -- Current behaviour is this: user has divert selected and wants to remove it. So he click on Delete. But hawtio console does not recognize divert and tries to remove address at which the divert is created. > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16386695#comment-16386695 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172316744 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js --- @@ -231,6 +234,17 @@ var ARTEMIS = (function(ARTEMIS) { } }); + workspace.subLevelTabs.push({ --- End diff -- If the intent is not to delete a divert as per reply to previous comment, then this should not be added > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16386697#comment-16386697 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172316762 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js --- @@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) { .when('/artemis/deleteAddress', { templateUrl: ARTEMIS.templatePath + 'deleteAddress.html' }) + .when('/artemis/deleteDivert', { --- End diff -- If the intent is not to delete a divert as per reply to previous comment, then this should not be added > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16385821#comment-16385821 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172121469 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html --- @@ -0,0 +1,24 @@ + + --- End diff -- Eg there is code in this pr that signifies otherwise > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16385819#comment-16385819 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172121106 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html --- @@ -0,0 +1,24 @@ + + --- End diff -- If that’s the case please close this pr > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16385786#comment-16385786 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user stanlyDoge commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172115995 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html --- @@ -0,0 +1,24 @@ + + --- End diff -- Hi Michael. After discussion with @andytaylor we decided that divert should not be deletable from web console, so there is no need to have pop-up. > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16385563#comment-16385563 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172086640 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html --- @@ -0,0 +1,24 @@ + + --- End diff -- Please see other delete's note the extra are you sure kind of pop up. This should be the same, see https://github.com/stanlyDoge/activemq-artemis/blob/0809b7794e16cd8981209f30f7ec3adf6ce0a114/artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteAddress.html > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16385558#comment-16385558 ] ASF GitHub Bot commented on ARTEMIS-1715: - Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1913#discussion_r172086296 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js --- @@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) { .when('/artemis/deleteAddress', { templateUrl: ARTEMIS.templatePath + 'deleteAddress.html' }) + . when('/artemis/deleteDivert', { --- End diff -- space between . and when > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console
[ https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16382062#comment-16382062 ] ASF GitHub Bot commented on ARTEMIS-1715: - GitHub user stanlyDoge opened a pull request: https://github.com/apache/activemq-artemis/pull/1913 ARTEMIS-1715 Disable to remove a divert from hawtio console You can merge this pull request into a Git repository by running: $ git pull https://github.com/stanlyDoge/activemq-artemis ARTEMIS-1715 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/activemq-artemis/pull/1913.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1913 commit 0809b7794e16cd8981209f30f7ec3adf6ce0a114 Author: Stanislav Knot Date: 2018-03-01T14:29:22Z ARTEMIS-1715 Disable to remove a divert from hawtio console > Disable to remove a divert from hawtio console > -- > > Key: ARTEMIS-1715 > URL: https://issues.apache.org/jira/browse/ARTEMIS-1715 > Project: ActiveMQ Artemis > Issue Type: Bug >Reporter: Stanislav Knot >Assignee: Stanislav Knot >Priority: Major > > Removing divert is broken (broker tries to remove diverted address, not > divert itself). It should be disabled to remove divert. -- This message was sent by Atlassian JIRA (v7.6.3#76005)