search

PacoVK / sqs-admin

A simple UI for local SQS
Apache License 2.0
51 stars 6 forks source link

UI doesn't support a large list of queues #255

Closed klutzer closed 1 year ago

klutzer commented 1 year ago

Hello man!

Thanks again for the PR before.

Another thing I've noticed is when we work with a number of queues greater than 3: The tab list isn't filled up. I think we could probably use cards or even a list/table similar to elasticmq interface, wdyt?

image

PacoVK commented 1 year ago

I like the idea 👍 The UI should always display the current messages in the queue. Therefore maybe we could have a sidebar that contains a list of the queues and the messages are still available in the main frame.

I'm a bit short in time currently, but i may have a look into it soon.

klutzer commented 1 year ago

Maybe we can fetch the messages from just the non-collapsed queues, in case of collapsed cards, or, as you said, having a sidebar with all the queues and keep the messages for the selected one in the main frame

PacoVK commented 1 year ago

@klutzer i created PR with a drawer component, this may fix the issue

klutzer commented 1 year ago

Nice! I've tested and seems ok! I would just include the messageGroupId and messageDeduplicationId attributes when showing the messages from FIFO queues

PacoVK commented 1 year ago

Good point, i released v0.3.0 FIFO messages now show the messageGroupId and messageDeduplicationId attributes. Thanks for conributing