Re: Absolute links in reply mails

2015-08-13 Thread Zach
Hey Chris, I posted a new issue to the google group, is that what you were asking for? Thanks for the quick response. -Zach On Wednesday, August 12, 2015 at 3:54:57 PM UTC-4, Christian Hammond wrote: Hi Zach, Can you file a bug report with a repro case to help us track this? It otherwise

Re: Absolute links in reply mails

2015-08-13 Thread Christian Hammond
Sorry, no, we need it posted on httpd://www.reviewboard.org/bugs/ Thanks! Christian On Thursday, August 13, 2015, Zach zach...@gmail.com wrote: Hey Chris, I posted a new issue to the google group, is that what you were asking for? Thanks for the quick response. -Zach On Wednesday,

Re: Absolute links in reply mails

2015-08-13 Thread Zach
Posted to https://code.google.com/p/reviewboard/issues/detail?id=3944 On Thursday, August 13, 2015 at 2:52:18 PM UTC-4, Christian Hammond wrote: Sorry, no, we need it posted on httpd://www.reviewboard.org/bugs/ Thanks! Christian On Thursday, August 13, 2015, Zach zac...@gmail.com

Re: Absolute links in reply mails

2015-08-12 Thread Zach
Hey Chris, We are actually having this issue again in reviewboard 2.0.18, except with file uploads. The src attribute of the image contains a relative link, the same as it is when you visit the actual review request (but the other attributes are different), eg: img class=modified-image

Re: Absolute links in reply mails

2015-08-12 Thread Christian Hammond
Hi Zach, Can you file a bug report with a repro case to help us track this? It otherwise will just get lost in the mailing list. Thanks :) Christian -- Christian Hammond - chip...@chipx86.com Review Board - https://www.reviewboard.org Beanbag, Inc. - https://www.beanbaginc.com On Wed, Aug

Absolute links in reply mails

2010-03-04 Thread Paul
My users are complaining because reply mails that have comments to lines of code end up with relative URLs rather than absolute URLs. e.g. a href=/r/42/diff/1/?file=173#file173line549 style=color: black; font-weight: bold; text-decoration: underline;/ops/configuration/ cfengine/inputs/cf.main/a

Re: Absolute links in reply mails

2010-03-04 Thread Christian Hammond
That patch should work, provided the correct variables are passed to the template renderer. I don't know if they are. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Thu, Mar 4, 2010 at 12:37 PM, Paul