Opened 16 years ago

Closed 4 years ago

#19250 closed enhancement (wontfix)

Modify SVN-Notify to add additional links into repository from the commit email

Reported by: jberry@… Owned by: admin@…
Priority: Normal Milestone:
Component: server/hosting Version:
Keywords: Cc:
Port:

Description

The commit emails put out by SVN-Notify would be more useful if they included links directly to the file in SVN or Trac.

A heading is put out before each diff section, which already includes path information and revisions. It seems like it would be nice if each of those were a true link into the trac representation.

The following heading, for instance, could include three links (one to the file in trac, one to the starting revision, and one to the ending revision).

Modified: trunk/base/src/port1.0/portlint.tcl (49405 => 49406)

Change History (7)

comment:1 Changed 16 years ago by wsiegrist@…

Status: newassigned

I think this is doable with a custom filter, probably implenting the diff subroutine.

SVN::Notify::Filter

comment:2 Changed 16 years ago by wsiegrist@…

Milestone: Website & Documentation
Type: defectenhancement
Version: 1.7.1

comment:3 Changed 10 years ago by jmroot (Joshua Root)

Owner: changed from wsiegrist@… to admin@…
Status: assignednew

comment:4 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from admin@… to admin@…
Status: newassigned

Since we moved to GitHub, we no longer use Subversion or SVN::Notify.

Not sure if the new mailer solution we're using offers such customizability.

comment:5 Changed 8 years ago by neverpanic (Clemens Lang)

Well, we have the link to the commit on GitHub, which has the 'View' button at the top right for every modified file. That's not directly in the mail, though.

comment:6 Changed 8 years ago by raimue (Rainer Müller)

The rest of the diff is just a the output of git diff with colors. Putting additional links into this to is not trivial. I would leave the current notifications produced by git-multimail as-is.

comment:7 Changed 4 years ago by raimue (Rainer Müller)

Resolution: wontfix
Status: assignedclosed

I think the current notification mails are good enough in this regard. Any changes to the diff output should be handled with git-multimail.

Note: See TracTickets for help on using tickets.