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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dataset ID in Changes graph tooltip is .../0, but one in the list below graph is .../1 #2093

Open
jhutar opened this issue Oct 17, 2024 · 1 comment
Assignees
Labels
area/UI type/bug Something isn't working

Comments

@jhutar
Copy link

jhutar commented Oct 17, 2024

Describe the bug

Dataset ID in Changes graph tooltip says Change in run .../0, but one in the list below graph says .../1

image

To Reproduce

https://horreum.corp.redhat.com/test/306?fingerprint=%7B%22.metadata.scenario.name%22%3A%22mvp%22%7D&end=1729116000000

Version

Current in Prod

Browser

firefox-130.0.1-1.fc40.x86_64

@lampajr lampajr added type/bug Something isn't working area/UI labels Oct 19, 2024
@lampajr
Copy link
Member

lampajr commented Oct 21, 2024

Hi @jhutar,

This is kind of intended (at least done on purpose) as you can see

<NavLink to={`/run/${dataset.runId}#dataset${dataset.ordinal}`}>
{dataset.runId}/{dataset.ordinal + 1}
</NavLink>
but I agree that this could be misleading.

My guess is that, initially, the idea was to keep Dataset ordering starting from 1 in the UI (whereas the internal ordering, in the backend, starts from 0).

@johnaohara I think we can simply remove that ordinal + 1 and show in the UI <dataset>/0 to avoid any confusion, anything against this?

If we proceed with this change, I think we should also apply the same in the datasets tab
image

[edit] Created a proposal here #2106

@lampajr lampajr self-assigned this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/UI type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants