copr icon indicating copy to clipboard operation
copr copied to clipboard

frontend: provide log-detective explain integration for failed builds

Open nikromen opened this issue 11 months ago • 7 comments

Merge after https://github.com/fedora-copr/logdetective-website/pull/207

nikromen avatar Jan 28 '25 15:01 nikromen

Would you mind uploading the screenshot?

praiskup avatar Feb 07 '25 07:02 praiskup

button next to the fail state:

srpm failure srpm

rpm failure rpm

or https://github.com/fedora-copr/copr/pull/3608/commits/793ee5b5bf560a6330b9317ae6eb301326e5eab1 introduces separate column for the table

separated column when rpm build fails sloupecek2

no separated column created when srpm fails srpm_no_sloupecek

nikromen avatar Feb 08 '25 23:02 nikromen

IMHO the buttons come out too aggressive. I would probably go with a normal link and an icon (to look the same as the chroot name in the first column). And I would probably shorten the text to something like "Explain failure" or "Explain with AI".

FrostyX avatar Feb 09 '25 13:02 FrostyX

like this?

kubas-feedback

nikromen avatar Feb 10 '25 11:02 nikromen

From the meeting: [x] failed (Ask AI / Teach AI)

praiskup avatar Feb 10 '25 12:02 praiskup

Pull Request validation

Failed

🔴 Review - Missing review from a member (2 required)

Success

🟢 CI - All checks have passed

github-actions[bot] avatar Feb 10 '25 13:02 github-actions[bot]

so here is every UI implementation introduced in this PR

please vote for your favorite one with emoji :D

vote with :tada: for buttons srpm

rpm

vote with :heart: for separate column and button in https://github.com/fedora-copr/copr/commit/793ee5b5bf560a6330b9317ae6eb301326e5eab1

sloupecek2

and this is how srpm fail looks srpm_no_sloupecek

vote with :eyes: for links instead of buttons in 4082872efb99326434a8edb10ef2eae15fc18de3

kubas-feedback

vote with :rocket: for Teach AI / Ask AI in separate column introduced in 12f0f552a2217cb885a698668b823090cb23773c. NOTE: this also deletes the green Log Detective button at the top since Teach AI replaces that function

2

and srpm fail 1

nikromen avatar Feb 24 '25 10:02 nikromen