activemq
activemq copied to clipboard
[AMQ-8274] Allow url parameters to specify initial sort order
This change will allow to pass sortColumnId
and sortOrder
url parameters to console's queues view, thus ordering the data on-load. This is very useful when visually monitoring dequeued messages change over time.
Hello @maxfortun, thanks for the PR. Will review. Do you happen to have a JIRA to reference for this?
If not, it is quick to open one at Apache JIRA -- select 'ActiveMQ' as the project and 'Web Console' as the component. Release notes are generated from the JIRAs, so features and fixes like this can be communicated to users.
Thanks!
Thank you, just created the issue:
https://issues.apache.org/jira/browse/AMQ-8274
On May 21, 2021, at 12:42 PM, Matt Pavlovich @.***> wrote:
Hello @maxfortun https://github.com/maxfortun, thanks for the PR. Will review. Do you happen to have a JIRA to reference for this?
If not, it is quick to open one at Apache JIRA https://issues.apache.org/jira -- select 'ActiveMQ' as the project and 'Web Console' as the component. Release notes are generated from the JIRAs, so features and fixes like this can be communicated to users.
Thanks!
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/apache/activemq/pull/655#issuecomment-846094142, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACNVZ5GTQFNF566I4ND2AODTO2EN3ANCNFSM45JPH4IQ.
I am not sure why maven tests are failing. My changes were only to a front end javascript file. Is this a blocker for a merge?
@maxfortun The tests probably failing due to previous Jenkins build rules. If you rebase the PR, it should be able to go green.
How do I get this PR merged in?
I’m working on tasks to get into 5.16.4. This is one. Should be within the next week. Thanks!
Would this be merged into 5.17 as well?
I will yes.