Re: gerrit-2.14 + gitiles-548e893(stable-2.14) produces wrong link in new ui
24 views
Skip to first unread message
Alon Bar-Lev
Apr 27, 2017, 2:48:51 AM4/27/17
to repo-d...@googlegroups.com
On 27 April 2017 at 00:22, Alon Bar-Lev <alon....@gmail.com> wrote:
>
> Hi,
>
> When trying to view a link in a merged change the following URL is produced:
>
> http://ilvpbg1252.corp.amdocs.com:8080/c/209/plugins/gitiles/xxx/+/51f6502a034b336a42aa33b31ad514a6543abba7
>
> Notice the /c/209 prefix which should be removed, I guess this link should be absolute and not relative.
>
> <a class="webLink style-scope gr-change-metadata" rel="noopener" target="_blank" href="plugins/gitiles/xxx/+/51f6502a034b336a42aa33b31ad514a6543abba7">
> browse
> </a>
>
> Regards,
> Alon
Here is the fix[1], although it is trivial, I am unsure if it is correct.
[1] https://gerrit-review.googlesource.com/c/104783/
>
> Hi,
>
> When trying to view a link in a merged change the following URL is produced:
>
> http://ilvpbg1252.corp.amdocs.com:8080/c/209/plugins/gitiles/xxx/+/51f6502a034b336a42aa33b31ad514a6543abba7
>
> Notice the /c/209 prefix which should be removed, I guess this link should be absolute and not relative.
>
> <a class="webLink style-scope gr-change-metadata" rel="noopener" target="_blank" href="plugins/gitiles/xxx/+/51f6502a034b336a42aa33b31ad514a6543abba7">
> browse
> </a>
>
> Regards,
> Alon
Here is the fix[1], although it is trivial, I am unsure if it is correct.
[1] https://gerrit-review.googlesource.com/c/104783/
Reply all
Reply to author
Forward
0 new messages