cockpit icon indicating copy to clipboard operation
cockpit copied to clipboard

Clickable BZ links in update changelog

Open supakeen opened this issue 3 years ago • 17 comments

On the updates page the last item from the changelog is shown, this is super nice. Often times RHBZ tickets are listed in these changelogs or other metadata. It'd be super nice if #\d+ could be replaced with a link to the relevant issue.

supakeen avatar Sep 21 '22 07:09 supakeen

I'd be happy to provide a PR if someone could point me in the right direction, I'm not that knowledgeable on the entire ecosystem surrounding cockpit :)

supakeen avatar Sep 21 '22 08:09 supakeen

Good idea!

It would need to be limited to distributions using the RH bugzilla (RHEL, Fedora, CentOS, etc.). That is, it probably shouldn't link to redhat.bugzilla.com in Debian, Ubuntu, SUSE, Arch, etc. distros.

garrett avatar Sep 21 '22 09:09 garrett

It might be around here: https://github.com/cockpit-project/cockpit/blob/main/pkg/packagekit/updates.jsx#L320

If you need help how to run and test cockpit please feel free to reach out!

marusak avatar Sep 21 '22 10:09 marusak

Thank you @marusak I'll take a look, @garrett is also absolutely right on the distro-conditional.

supakeen avatar Sep 21 '22 10:09 supakeen

Is this still open? I would like to work on this issue.

ayeankit avatar Mar 03 '23 10:03 ayeankit

Since I don't have any spare time at the moment feel free @ayeankit.

supakeen avatar Mar 03 '23 11:03 supakeen

Hello,

I'm Deep. I'm interested in working on this issue. Could you please explain the problem and guide me on setting up the repository on my computer? I'm eager to contribute and give my best effort to resolve this. This will be my first valuable contribution. Thank you for giving me this opportunity.

Deeppjp116 avatar May 30 '23 03:05 Deeppjp116

As explained above: Links in changelogs that go to bugzilla numbers should be linked to the distribution's bugzilla. This is different for each distribution... and not all use Bugzilla.

Setting up Cockpit for development is detailed on our website:

  • Page with all the dev docs: https://cockpit-project.org/external/wiki/Contributing.html
  • Getting up and running; how to install Cockpit for development locally: https://cockpit-project.org/external/source/HACKING.html

garrett avatar Jun 12 '23 14:06 garrett

Can i work on this issue?

ashutosh7i avatar Dec 04 '23 20:12 ashutosh7i

@ashutosh7i sure, please do, thanks! This should be relatively easy to do. Note that these days, RHEL actually primarily uses https://issues.redhat.com , but changelogs could still refer to bugzilla. It usually looks like "rhbz#12345" or "jira#12345".

martinpitt avatar Dec 05 '23 05:12 martinpitt

ok so i was wondering how to reproduce the changelogs?

as the modification has to be done with changelogs or other metadata. in my cockpit dashboard i am not having any, is there any way to reproduce these or anything else i am missing?

output2

ashutosh7i avatar Dec 05 '23 13:12 ashutosh7i

Our tests mock the metadata, e.g. here: https://github.com/cockpit-project/cockpit/blob/main/test/verify/check-packagekit#L740 I suggest to add a testlib.sit() at the relevant point, and run the test with TEST_SHOW_BROWSER=1 TEST_BROWSER=firefox, then you can interactively work with that (including ./build.js -w -r c)

martinpitt avatar Dec 05 '23 13:12 martinpitt

@martinpitt Hello, Can i work on this issue ?

kh-ac avatar Dec 18 '23 14:12 kh-ac

@kh-ac sure, please do!

martinpitt avatar Dec 18 '23 14:12 martinpitt

@supakeen I am new to this project , can you explain little bit more about the issue,I would love to work on it.

chiteshbansal avatar Jan 03 '24 03:01 chiteshbansal

@chiteshbansal In the packages listing in Cockpit (forgive me if I get terminology wrong) you often see references to Bugzilla tickets by their number usually in the format of BZ#1234 or RHBZ#1234, JIRA#1234 or such. I'd like those to be clickable so I can go directly to the issue.

Note that RHBZ links to the RedHat Bugzilla and JIRA links to the RedHat JIRA.

supakeen avatar Jan 03 '24 07:01 supakeen

@martinpitt

Our tests mock the metadata, e.g. here: https://github.com/cockpit-project/cockpit/blob/main/test/verify/check-packagekit#L740 I suggest to add a testlib.sit() at the relevant point, and run the test with TEST_SHOW_BROWSER=1 TEST_BROWSER=firefox, then you can interactively work with that (including ./build.js -w -r c)

I am a beginner, so this question might be dumb . Can you give some more details about this,on how to generate those changelogs?

mega-bots avatar Jan 25 '24 13:01 mega-bots