Allow links in reason text

[reuben]

Currently the reason text is escaped and shows as <a href…

[edmorley]

We'll need to preserve the use case mentioned in bug 846868 before we merge this.

[catlee]

As long as tbpl is getting the reasons/motd via the API and not via the
rendered HTML, we should be able to linkify treestatus' UI, right?

On Wed, Mar 6, 2013 at 8:42 AM, Ed Morley notifications@github.com wrote:

We'll need to preserve the use case mentioned in bug 846868 before we
merge this.

—
Reply to this email directly or view it on GitHubhttps://github.com/catlee/treestatus/pull/33#issuecomment-14499664
.

[edmorley]

That's not the use case; sheriffs repopulating the treestatus UI using old statuses is.

[reuben]

A better solution would be to have a "restore" button next to statuses, but we could not linkify if the user is a sheriff. Is that OK?

[edmorley]

A restore button sgtm :-)

[reuben]

Looks like this: http://cl.ly/image/1m033f3V3k1a
I decided to just populate the fields since that allows sheriffs to edit it before setting the status.

[edmorley]

That looks great :-)

I don't have permissions on this repo (and catlee will need to sign off anyway), so I'll leave the rest to him.

Thank you for doing this!

[catlee]

If bleach is a pure python library, then it should be put into the vendor lib. Does it have any compiled libraries?