#6837 imported trac content with r123 refs still link to old host

v1.1.0
closed
Cory Johns
trac (14) import (116) sf-2 (994)
General
Dave Brondsema
2015-08-20
2013-11-04
Dave Brondsema
No

E.g. :

Also fixed for ExecutePipeline? in [r10739](http://sourceforge.net/apps/trac/open-ms/changeset/10739).

I think we want to rewrite this to simply [r10739] or possibly [r10739](r10739) (so that brackets aren't displayed).

IIRC, there are more specific similar syntaxes too that may need to be considered.

Discussion

  • Cory Johns - 2013-11-15
    • status: open --> in-progress
    • assigned_to: Cory Johns
     

  • Dave Brondsema

    Dave Brondsema - 2013-11-15
    • Milestone: forge-nov-15 --> forge-nov-29
     

  • Cory Johns - 2013-11-15
    • Size: --> 2
     

  • Cory Johns - 2013-11-15
    • status: in-progress --> code-review
     

  • Cory Johns - 2013-11-15

    allura:cj/6837

     

  • Dave Brondsema

    Dave Brondsema - 2013-11-18
    • QA: Dave Brondsema
     

  • Dave Brondsema

    Dave Brondsema - 2013-11-18

    The linking changes look great. Would it be easy to make a change for the wiki links that have the ? appended? They look like <a class="missing wiki">RedHat?</a> on https://sourceforge.net/apps/trac/mcclidar/ticket/47 for example. The don't come through as a link (since there's no href) which is fine, but the extra ? isn't good especially since its just plain text and not styled as a possible-link any more.

     

  • Cory Johns - 2013-11-18

    That would have to be handled at a higher level than these changes, but it shouldn't be too hard with the changes from [#6836]. I'll take a look.

     

    Related

    Tickets: #6836

  • Cory Johns - 2013-11-18

    Rebased and force-pushed

     

  • Dave Brondsema

    Dave Brondsema - 2013-11-18
    • status: code-review --> closed
     

Log in to post a comment.