Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Breadcrumbs show the UUID rather than asset name #4226

Open
lache-melvin opened this issue Jun 21, 2024 · 5 comments
Open

Breadcrumbs show the UUID rather than asset name #4226

lache-melvin opened this issue Jun 21, 2024 · 5 comments
Labels
馃洃 Blocked bug Something is borken Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release.

Comments

@lache-melvin
Copy link
Contributor

lache-melvin commented Jun 21, 2024

What went wrong? 馃槻

Slowed down for your viewing pleasure... it does get updated, and then overwritten!

equipment.breadcrumb.mov

Expected behaviour 馃

Asset/Immunizations name stays in the breadcrumb!

How to Reproduce 馃敤

Steps to reproduce the behaviour:

  1. Go to Coldchain > Equipment
  2. Click on an asset to go to the detail view
  3. See breadcrumb: UUID!

Your environment 馃尡

  • Open mSupply Version: 2.1 RC1
  • Legacy mSupply Central Server Version:
  • Platform: browser (chrome)
  • Database type:
@lache-melvin lache-melvin added bug Something is borken needs triage needs daily triage New bug to be triaged by the release team labels Jun 21, 2024
@lache-melvin
Copy link
Contributor Author

WEIRDLY it works when you navigate from list view to detail page. But when you refresh the page, it goes back to being a uuid 馃憖

@mark-prins
Copy link
Collaborator

hm.. a re-render maybe, and the suffix call isn't made on the second one? is it worth refactoring the breadcrumb and see if that fixes this?

@CarlosNZ
Copy link
Contributor

Existing related issues:

As Mark said, we should probably tackle that refactor and then see what remains of this problem.

@CarlosNZ CarlosNZ removed the needs daily triage New bug to be triaged by the release team label Jun 23, 2024
@marthakeezy marthakeezy changed the title CC Equipment breadcrumb shows the UUID rather than asset name Breadcrumbs show the UUID rather than asset/immunization name Jun 23, 2024
@marthakeezy
Copy link

Just updated the issue to include another occurrence under Programs -> Immunizations on RC-2 :)

@mark-prins mark-prins added 馃洃 Blocked Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. and removed needs triage labels Jun 27, 2024
@andreievg
Copy link
Collaborator

andreievg commented Jun 27, 2024

Triage, putting this on hold, setting higher priority to this: #2118, once the refactor is done we can re-visiting this and re-check. Refactor to be prioritised for this RC, @mark-prins suggest it's only a couple of hours to do, thus should be easy to validate and test in RC

@jmbrunskill jmbrunskill changed the title Breadcrumbs show the UUID rather than asset/immunization name Breadcrumbs show the UUID rather than asset name Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
馃洃 Blocked bug Something is borken Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release.
Projects
None yet
Development

No branches or pull requests

5 participants