Hi Chris,

I've updated the patch that contains all the suggestions. I think the patch is ready to merge.

One thing that I would like to ask you about the package and package-name in web/repository/controller.scm.

When I test the URL below I'm getting this error. (https://pastebin.ubuntu.com/p/HdKShmKqH7/)

('GET "repository" repository-id "branch" branch-name "package" package-name) -> http://localhost:8765/repository/1/branch/master/package/emacs

What do you think? BTW it's accessible on the official server.

https://data.guix.gnu.org/repository/1/branch/master/package/emacs/

Could you please review the patch attached?
I'm very excited to make my first FOSS contribution. :)

Thanks for your help,
Canan Talayhan

On Fri, Apr 16, 2021 at 2:11 PM Christopher Baines <mail@cbaines.net> wrote:

Canan Talayhan <canan.t.talayhan@gmail.com> writes:

> After your suggestions, I've fixed the patch. But for the revision
> part, I have a question. You said "I'd put "Channel News Entries"
> first, then the Revision bit second." There are so many titles like
> this structure in Revision views.
>
> For example :
> ->System tests Revision c7d0441
> or
> ->Revision c7d0441 System tests
>
> Should I fix all of them this way? Or just for Channel News Entries?

I think having the System tests bit first is better, that way different
pages about the same revision will have more distinct titles, and yeah,
I'd apply the same principle to other titles as well.