dashboard icon indicating copy to clipboard operation
dashboard copied to clipboard

Incorrect app bar cluster abbreviations

Open richard-cox opened this issue 1 year ago • 5 comments

Setup

  • Rancher version: pre 2.8.0

Describe the bug

  • rc-rke1-3 and rc-rke1-4 both shorten to RR1 instead of RR3 and RR4

To Reproduce

  • create imported clusters with names above
  • view them in the new app bar

Result image

Expected Result RR3 and RR4

richard-cox avatar Sep 29 '23 14:09 richard-cox

In addition we should look at how these work

https://github.com/rancher/dashboard/blob/master/shell/utils/tests/cluster.test.ts#L36C1-L43C44

    ['customer-support-team-1', 'cst'],
    ['customer-support-team-one', 'cst'],
    ['customer-support-team-prod', 'cst'],
    ['customer-support-team-dev', 'cst'],
    ['customer-support-team-prod-1', 'cst'],
    ['customer-support-team-dev-1', 'cst'],
    ['customer-support-team-prod-2', 'cst'],
    ['customer-support-team-dev-2', 'cst'],

richard-cox avatar Oct 04 '23 09:10 richard-cox

We should tackle https://github.com/rancher/dashboard/issues/9840 at the same time

richard-cox avatar Jan 29 '24 21:01 richard-cox

For reference, here are the specs for the cases with & without cluster colour:

Adobe Xd Specs

edenhernandez-suse avatar Apr 30 '24 10:04 edenhernandez-suse

I didn't spot a check for this test expectation:

  • Opening the Customise modal, the Use custom badge & Apply buttons should be disable.

Let me know if I've missed it but I checked the e2e and I didn't find an is disabled assertion. @yonasberhe23

izaac avatar May 02 '24 17:05 izaac

@nwmac with @scures gone, can you answer this question from QA?

gaktive avatar May 16 '24 17:05 gaktive

Opening the Customise modal, the Use custom badge & Apply buttons should be disable.

@izaac This has partially changed during the lifetime of the PR. the use custom badge should be enabled and the Apply button disabled. However this isn't working (when there are no setting the Apply button is enabled). I think there were some other changes we might make as well, so i createed a separate issue

richard-cox avatar May 20 '24 08:05 richard-cox

Thank you @richard-cox

nwmac avatar May 20 '24 08:05 nwmac

I think it's safe to move this to done. test coverage is sufficient with e2e testing the modal functionality and unit testing the cluster abbreviations. @izaac

yonasberhe23 avatar May 20 '24 20:05 yonasberhe23