artemiscloud / activemq-artemis-self-provisioning-plugin

ActiveMQ Artemis Self Provisioning Plugin
https://main--633c221f1c3e24f3029a826f.chromatic.com/
Apache License 2.0
3 stars 12 forks source link

[#294] Added 'Brokers' entry within the Workloads section #302

Closed Msarawan closed 2 months ago

Msarawan commented 2 months ago

Since Brokers are a type of workload, a shortcut to our project has been added within the Workloads section as 'Brokers' which displays a list of brokers.

fixes: #294

Msarawan commented 2 months ago

When I'm selecting the brokers, I can see the proper page. However two sections are getting selected:

image

Can you update the contribution so that, when I click on Brokers, only this entry gets selected in the page?

Alternatively, if that isn't possible, let's completely remove the ActiveMQ Artemis entry.

@RoddieKieley do you have an opinion on this?

Hello @lavocatt , about the selection of both the entries i will take a look. but, we can't remove the entry with Activemq Artemis because in future we need to update this name with Messaging on openshift. so i think we cannot remove this entry as per our discussion atm. but will confirm from Roderick.

lavocatt commented 2 months ago

Hey @Msarawan after checking with @RoddieKieley, the correct approach is to remove the ActiveMQ Artemis entry and keep only the Brokers one under Workloads

Msarawan commented 2 months ago

Hey @Msarawan after checking with @RoddieKieley, the correct approach is to remove the ActiveMQ Artemis entry and keep only the Brokers one under Workloads

ohh okay!, thanks for the clarification. I will update now.