Actually when pressing on "Make a donation" it opens https://donate.mozilla.org/about/ This should be redirected now to https://donate.mozilla.org/thunderbird/about/ In aboutDialog.xul the link is https://sendto.mozilla.org/page/contribute/givenow-seq?source=thunderbird_about I don't know where this bug should be filed best. So I'll cc rkent.

Richard, can you attach a patch. Since Kent is doing uplifts on Monday and the matter is supposedly urgent, maybe he wants to include this. Also, could you please add a second patch to "Donate" to the help menu, maybe under "What's New". You're the UX guy, so you'll know where. BTW, on April 22, 2016 I wrote on TB drivers on the issue of redirecting "What's New": https://bugzilla.mozilla.org/show_bug.cgi?id=1266795 BTW, why don't we do a "Donate" menu item in that section as well? (but that got ignored).

I'm not sure we need a patch. I think this is more a config of a redirect on the web server.

Right, but we'd need one for the "Donate" menu item. But that's a string change. Or maybe not, since we already have "Make a donation".

Let's just change the URL, I'll take a look. Regarding the Help menu, my work in bug 1134760 adds a link, but the patch got stuck since I don't have a mac and the rework causes some problem on mac that would have to be figured out locally :(

(In reply to Magnus Melin from comment #4) > Let's just change the URL, I'll take a look. > > Regarding the Help menu, my work in bug 1134760 adds a link, but the patch > got stuck since I don't have a mac and the rework causes some problem on mac > that would have to be figured out locally :( Indeed. There's also Appmenu. And there might even be another bug report floating around somewhere - pretty sure I saw something in the past week. "Get Involved" also needs to change. We've been using https://wiki.mozilla.org/Thunderbird#Contributing

(In reply to Magnus Melin from comment #4) > Let's just change the URL, I'll take a look. > Regarding the Help menu, my work in bug 1134760 adds a link, ... OB, but what's wrong with adding a simple menu item for now using the existing strings? "Make a donation"

The localized string for that is not likely to work out for all localizations, it may be totally wrong and look the whole menu look weird.

Ideally we should not hardcode en-US but leave that out and let the server redirect as appropriate. I notice that's not completely working out atm though. https://donate.mozilla.org/thunderbird/ -> https://donate.mozilla.org/en-US/thunderbird/ (correct) https://donate.mozilla.org/thunderbird/about/ -> https://donate.mozilla.org/en-US/about/ (wrong: the Mozilla general donation page)

(In reply to Richard Marti (:Paenglab) from comment #0) > Actually when pressing on "Make a donation" it opens > https://donate.mozilla.org/about/ This should be redirected now to > https://donate.mozilla.org/thunderbird/about/ > > In aboutDialog.xul the link is > https://sendto.mozilla.org/page/contribute/givenow- > seq?source=thunderbird_about > > I don't know where this bug should be filed best. So I'll cc rkent. https://donate.mozilla.org/en-US/thunderbird/about/ works as does https://donate.mozilla.org/foobar/thunderbird/about/ https://donate.mozilla.org/thunderbird/about/ puts you on the generic Mozilla donate page, which of course is not what we want. I assume there must be some way to insert the Thunderbird locale into this location, and it should redirect to en-US if there is no locale defined. Yet it is important that we get a locale properly defined since Germany and Japan are our top countries. Bugs are filed at https://github.com/mozilla/donate.mozilla.org/issues with the Thunderbird tag.

Comment on attachment 8752609 [details] [diff] [review] bug1272972_about_donate-link.patch Review of attachment 8752609 [details] [diff] [review]: ----------------------------------------------------------------- Really the correct change should pull the LOCALE in somehow, such as is done from all-thunderbird.js in various locations. But this change works, and given that the donate site does not have any current localization, works for now. So let's use it in the short run. But it would be good to file a followup patch to change this to include a locale, so that in the future when we get the donate links localized, they will work correctly.

I was wrong about translation. This gives a German translation for example: https://donate.mozilla.org/de/thunderbird/about/ Also Japanese: https://donate.mozilla.org/ja/thunderbird/about/ It would be great if we could get the translated link in place soon.

The German is pretty good, and so are these: https://donate.mozilla.org/fr/thunderbird/about/ https://donate.mozilla.org/es/thunderbird/about/ Wouldn't it just be href="https://donate.mozilla.org/%LOCALE%/thunderbird/about instead of href="https://donate.mozilla.org/en-US/thunderbird/about

Maybe not. Perhaps %LOCALE% only works in all-thunderbird.js. In this case, we just need to define a preference.

https://hg.mozilla.org/comm-central/rev/ed578f951bb8 -> FIXED Locale isn't usually hardcoded in these kind of links. You end up at the proper locale through server side redirect - firefox does that too for this case and it works out for Thunderbird too except for the subdir (thunderbird/about/) as I described comment 8.

Sorry, I never understood the (cryptic) comment #8. You coded: https://donate.mozilla.org/en-US/thunderbird/about/ So where does this go if A German, Japanese, French or Spanish user clicks this? To that very page, right? So why not invest the effort to make it respect the locale?

(In reply to Jorg K (PTO during summer, NI me) from comment #15) > So why not invest the effort to make it respect the locale? It will be respected, just the effort is better spent doing a server side redirect as that will of course be needed for links in general also. E.g. in social media you want to give the right language automatically.

While the web works isn't done, I think it's more useful to send people to the localised page. This would need to be applied on top of Magnus' patch.

Comment on attachment 8752730 [details] [diff] [review] Patch to redirect to https://donate.mozilla.org/%LOCALE%/thunderbird/about Sorry about the trailing space.

Comment on attachment 8752730 [details] [diff] [review] Patch to redirect to https://donate.mozilla.org/%LOCALE%/thunderbird/about I can confirm that this patch works (at least for my en-US locale) when compiled under esr45. And it would be good to get it into TB 45. It would be good to get Magnus's comments though.

Comment on attachment 8752609 [details] [diff] [review] bug1272972_about_donate-link.patch Since the second patch is pushed on top of this one, there is no reason we should wait to uplift this.

I'm leaving this open while we resolve the issue of the second patch. But this is very low risk, so I am willing to consider adding the second patch to TB 45.1.2 even if it misses the release candidate 45.2b1 From my perspective, while "Locale isn't usually hardcoded in these kind of links" is true and we are trying to resolve the MoFo bug that causes this, the first patch also hardcodes locale and is not the best solution after the server side issue is fixed. If we have to hardcode, we might as well hardcode the right locale!

Landings for uplift: http://hg.mozilla.org/releases/comm-aurora/rev/724410b84b45 http://hg.mozilla.org/releases/comm-beta/rev/404d60856521 http://hg.mozilla.org/releases/comm-esr45/rev/511ffae6b7be

I wonder if it's a good idea to have the donate link as a pref. There may be some greyzone cases where that would be taken advantage of, e.g. in general distributions can change prefs and still be allowed to use the brand name. Firefox also hardcodes it... Anyway, I looked at the donation page redirect. I found where the problem is but still need to figure out the proper fix.

We already have: pref("app.releaseNotesURL", ... pref("toolkit.crashreporter.infoURL", ... pref("app.support.baseURL", ...

(In reply to Magnus Melin from comment #24) > Firefox also hardcodes it... Yes, to https://sendto.mozilla.org/page/contribute/Give-Now?source=mozillaorg_default_footer&ref=firefox_about&utm_campaign=firefox_about&tm_source=firefox&tm_medium=referral&utm_content=20140929_FireFoxAbout That's what you just removed ;-) https://sendto.mozilla.org/page/contribute/givenow-seq?source=thunderbird_about

(In reply to Jorg K (PTO during summer, NI me) from comment #25) > We already have: > > pref("app.releaseNotesURL", ... > pref("toolkit.crashreporter.infoURL", ... > pref("app.support.baseURL", ... Yes but it makes no money for anyone so it's not comparable. IIRC some of those are in fact changed in say Ubuntu's Thunderbird version.

(In reply to Magnus Melin from comment #24) > I wonder if it's a good idea to have the donate link as a pref. There may be > some greyzone cases where that would be taken advantage of, e.g. in general > distributions can change prefs and still be allowed to use the brand name. > Firefox also hardcodes it... > > Anyway, I looked at the donation page redirect. I found where the problem is > but still need to figure out the proper fix. That's a good point.

Currently we open https://donate.mozilla.org/en-US/thunderbird/about/. Are you saying that with some website work that could be redirected to https://donate.mozilla.org/de/thunderbird/about/ https://donate.mozilla.org/ja/thunderbird/about/ https://donate.mozilla.org/fr/thunderbird/about/ https://donate.mozilla.org/es/thunderbird/about/ etc. by the website depending on where the visitor comes from?

The more like redirect starting place, which works now for the similar https://donate.mozilla.org/thunderbird, is: https://donate.mozilla.org/thunderbird/about

Comment on attachment 8752730 [details] [diff] [review] Patch to redirect to https://donate.mozilla.org/%LOCALE%/thunderbird/about Looks like we won't be using this patch in the end so I'm cancelling the review. It would be nice of someone could answer my question from comment #29 one day. Re. comment #30: https://donate.mozilla.org/thunderbird goes to a TB donation page. https://donate.mozilla.org/thunderbird/about goes to Mozilla donation pare.

"It would be nice of someone could answer my question from comment #29 one day." My comment 30 was supposed to be the answer. That is, the correct root redirect url should be: https://donate.mozilla.org/thunderbird/about

Continuing in bug 1277948 for easier tracking.