activemq icon indicating copy to clipboard operation
activemq copied to clipboard

[AMQ-8274] Allow url parameters to specify initial sort order

Open maxfortun opened this issue 3 years ago • 6 comments

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.

maxfortun avatar May 21 '21 16:05 maxfortun

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!

mattrpav avatar May 21 '21 16:05 mattrpav

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.

maxfortun avatar May 21 '21 16:05 maxfortun

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 avatar Nov 11 '21 16:11 maxfortun

@maxfortun The tests probably failing due to previous Jenkins build rules. If you rebase the PR, it should be able to go green.

mattrpav avatar Nov 11 '21 16:11 mattrpav

How do I get this PR merged in?

maxfortun avatar Jan 19 '22 13:01 maxfortun

I’m working on tasks to get into 5.16.4. This is one. Should be within the next week. Thanks!

mattrpav avatar Jan 19 '22 14:01 mattrpav

Would this be merged into 5.17 as well?

maxfortun avatar Sep 23 '22 17:09 maxfortun

I will yes.

jbonofre avatar Sep 23 '22 17:09 jbonofre