757209 - update YSI urls for creating accounts and learning more

Status: RESOLVED FIXED

(Thunderbird :: Message Compose Window, defect)

Description

[David :Bienvenu]

Attachment: proposed fix

YSI wants us to use

        "https://www.yousendit.com/thunderbird-about-yousendit?s=4001583&cid=pm-4001583" for learn more and "https://www.yousendit.com/litesignup?s=4001583&id=pm-4001583" for needAnAccount.

Comment 1

[David :Bienvenu]

we'll want to land this on aurora and beta asap.

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 625760 [details] [diff] [review]
proposed fix

Review of attachment 625760 [details] [diff] [review]:
-----------------------------------------------------------------

Besides the & problem, this looks good (though I didn't compile / test it...vacation 'n all. ;) )

-Mike

::: mail/components/cloudfile/content/YouSendIt/settings.xhtml
@@ +19,5 @@
>      <form id="provider-form" onsubmit="return false;">
>        <label for="username">&youSendItSettings.username;</label>
>        <input id="username" type="text" required="true"/>
>        <div id="learn-more" class="float-right">
> +        <a href="https://www.yousendit.com/thunderbird-about-yousendit?s=4001583&amp;cid=pm-4001583">&youSendItSettings.learnMore;</a>

I think the &amp; in the URL is wrong - I think that's supposed to be a pure &.  Same below.

Comment 3

[David :Bienvenu]

heh, if & worked, this would have taken me 2 minutes instead of a couple hours. I needed to use & to get the UI to not get a xml error.

I'll let you be back from vacation to try it out :-) No hurry, just wanted to get the process going...enjoy your day off!

Comment 4

[David :Bienvenu]

fixed on trunk - I'll push to alpha and beta.

Comment 5

[David :Bienvenu]

trunk - http://hg.mozilla.org/comm-central/rev/2e9cb0ffa579
aurora - http://hg.mozilla.org/releases/comm-aurora/rev/9ddcd204a766
beta - http://hg.mozilla.org/releases/comm-beta/rev/7599f7180908

beta didn't apply cleanly, so I had to transplant by hand.