-
-
Notifications
You must be signed in to change notification settings - Fork 383
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
Please add words that describe the event types in the dashboard #3182
Comments
Adding tooltips is a good idea, but anything else I wouldn't do. |
I am already updating the part 😉 |
I am also thinking about some general improvements to the list. Just tried to get some inspiration from v0: https://v0.dev/r/ezhlRyoM0sq 😅 What do you think about removing the commit sha from the list. |
FWIW, when there is an error having as much context available as possible makes things easier to diagnose and understand. I appreciate having things be compact and beautiful in the (hopefully common!) passing case, but when things are broken, if it is not possible to understand why, then the software is less useful. Most users of CI are not experts in the CI system itself, so expecting them to have instant recall and understanding of the features of the CI system is a big expectation, representing a massive learning curve.
Please don't thin down the information shown for these status lines. In fact, I would love it if there was a way to drill down into seeing more information.
Having words that can be copied and pasted is really useful. Having more context in the error message would also be incredibly valuable. eg: The So while my immediate concern would be helped with a tooltip, I would appreciate it if you all could consider making the diagnostics and data available in the UI quicker and easier for someone to act upon. |
I'm against this. Now, from commit I can say, that 82 and 81 are the same commit, 84 was force push of 83, 85 is just tagged 84.
In that case, we can probably move down commit message a bit and add PR header above: |
Clear and concise description of the problem
I had a difficult time understanding this failure; there's no written out words to indicate the event type and the icons are not intuitive enough to understand.
It turned out that this was neither a pr nor a push event, and the secrets were not allowed to be used with it.
Drilling into the event also showed no additional useful context beyond just the raw error condition:
Suggested solution
Please add some words in there somewhere!
For example, it would be really useful to see something like
event: deployment
or whatever this actual event was in the drilled down state.Please add tooltips to the icons to label the event type, so that hovering over it will spell out to the user what it is.
Alternative
No response
Additional context
No response
Validations
next
version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]The text was updated successfully, but these errors were encountered: