699: Automatically create hyperlinks in review description/comments

regis.d*******@gmai***** (Google Code) (Is this you? Claim this profile.)
chipx86
 chipx86
March 5, 2010 
> What's the URL of the page this enhancement relates to, if any?

https://reviewboard.eng.vmware.com/r/48041/

See the change's description: it contains URLs at the top.
See eknauft's very first comment: it contains the words "bug 283812".

> Describe the enhancement and the motivation for it.

I would like the URLs to be hyperlinks to their targets, I would like "bug 283812" to be an 
hyperlink to the bugzilla page.

This is very similar to what bugzilla does in bug description/comments, and it is very convenient.

Here is a short list of things that you guys could automatically hyperlink in review 
description/comments:

1) email addresses
2) actual URLs
3) "bug 12345", "PR #12345", "defect# 12345" should all link to the bugzilla page.
4) "CLN 12345", "change 12345", "@12345", should all link to the perforce page for the change 
(like tinderbox.eng.vmware.com does)

I'm sure you guys can figure out plenty of other things to automatically hyperlink by talking to 
agraveley, a.k.a. Mr TomBoy :)
david
#1 david
  • -Type-Defect
    +Type-Enhancement
    +Component-Reviews
chipx86
#2 chipx86 
Some of this is done. I'll get to e-mail addresses later (they work if using mailto:)
  • +Started
  • +chipx86
chipx86
#3 chipx86 
Most of this is done. We can possibly look into adding more things (e-mail addresses,
change numbers/revisions once we have some way of linking that) in 1.5.
  • -Started
    +Confirmed
  • +Milestone-Release1.5
#4 swta****@gmai***** (Google Code) (Is this you? Claim this profile.) 
When I click on one of the bug links in a changeset description, it opens the text
field for editing before navigating to the bug page. It doesn't affect the
functionality at all, but it is a little confusing/distracting.
chipx86
#5 chipx86 
New ones can be added as separate requests. I'm happy with what we have for now, and
I don't want this to be a tracker for everything involving auto-linking.
  • -Confirmed
    +Fixed