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.

Sign up for GitHub

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

Jump to bottom

🐛 : correct job numbering #141

Closed
juwit opened this issue Sep 18, 2019 · 0 comments · Fixed by #143
Closed

🐛 : correct job numbering #141

juwit opened this issue Sep 18, 2019 · 0 comments · Fixed by #143
Assignees
@juwit
Labels
🐛 bug Something isn't working ✨ feature:jobs Jobs 👤 user:end-user end-user
Milestone
1.3.0

Comments

@juwit
Copy link
Member

juwit commented Sep 18, 2019

Describe the bug
Job are ordered from the last to the first execution. This works well.
But the jobs are numbered using an array index, which is confusing (last job is numbered 1, first job X)

To Reproduce
Steps to reproduce the behavior:

  1. Go to to a stack having multiple jobs
  2. See job order and numbers

Expected behavior
Numbers should follow the right order (from latest to first)

Screenshots
image
In this screenshot, the first job in the list should be numbered 4.

Desktop (please complete the following information):

  • OS: Ubuntu 19.04
  • Browser: Firefox 69.0
  • Version: gaia 1.2.0

Additional context
N/A
@juwit juwit added 🐛 bug Something isn't working ✨ feature:jobs Jobs 👤 user:end-user end-user labels Sep 18, 2019
@juwit juwit self-assigned this Sep 23, 2019
@juwit juwit added this to the 1.3.0 milestone Sep 23, 2019
juwit added a commit that referenced this issue Sep 23, 2019
@juwit
🐛 : correct job numbering
e846485 
resolves #141
@juwit juwit mentioned this issue Sep 23, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@juwit juwit

Labels
🐛 bug Something isn't working ✨ feature:jobs Jobs 👤 user:end-user end-user
Projects
None yet
Milestone
1.3.0
Development

Successfully merging a pull request may close this issue.

🐛 : correct job numbering gaia-app/gaia
1 participant
@juwit