Closed Bug 1388914 Opened 7 years ago Closed 6 years ago

Delete thunderbird content from www.mozilla.org

Categories

(www.mozilla.org Graveyard :: Thunderbird, enhancement)

Product:
www.mozilla.org Graveyard
Component:
Thunderbird
Version:
Production
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: hoosteeno, Unassigned)

References

Details

Attachments

(1 file)

Link to GitHub pull-request: https://github.com/mozilla/bedrock/pull/5587
44 bytes, text/x-github-pull-request
Details | Review 
In bug 1363162 we see that thunderbird.net is ready to handle all marketing traffic for Thunderbird. Once bug 1388913 is complete we should delete thunderbird content from www.mozilla.org.
Component: Pages & Content → Thunderbird
The Thunderbird pages on mnozorg are now ready for removal, as per https://bugzilla.mozilla.org/show_bug.cgi?id=1363162#c8
Note: (from the other bug), we need a redirect: one rewrite of mozilla.org/thunderbird/* -> thunderbird.net/*.
Once the redirect is in place, we can run a report using DeepCrawl to find links pointing to */thunderbird/* that result in a 301, to drop in a follow-up hygiene bug.
Looks like the new thunderbird.net still points to https://www.mozilla.org/en-US/privacy/thunderbird/ so I will leave those pages and assets.
Yes, I haven't duplicated anything outside of mozilla.org/{lang}/thunderbird/*, so policy stuff, security advisories(https://www.mozilla.org/en-US/security/known-vulnerabilities/thunderbird/), and anything else outside of the main TB site should not be deleted. We're still relying on Mozilla Foundation for legal support/policy documents etc.
Cool, thanks for the confirmation Andrei.
There's also this page, related to the calendar above but I could not find an obvious place to redirect it: http://127.0.0.1:8000/en-US/projects/calendar/holidays/

Some of the calendars have expired (England only went to 2015 for example) but others will be good to 2030.
(In reply to Stephanie Hobson [:shobson] from comment #8)
> Could we redirect this as well?
> https://www.mozilla.org/en-US/projects/calendar/ ->
> https://addons.mozilla.org/en-US/thunderbird/addon/lightning/

The web page is more high level than the addons page.
But we could host it - /thunderbird/calendar ?
Flags: needinfo?(philipp)
I can duplicate the calendar content. www.thunderbird.net/calendar, seems like the logical place. /calendar/holidays for the holiday calendar page.

Will have to wait for next week though.
I don't know if you want to migrate the styleguide pages as well: https://www.mozilla.org/en-US/styleguide/identity/thunderbird/logo/

The team is planning to decommission the styleguide and just have links there pointing to other resources.
Should this page be removed as well? https://www.mozilla.org/en-US/thunderbird/all/
Flags: needinfo?(sancus)
Yes, you can remove /all/, it's under /thunderbird/* and we have it https://www.thunderbird.net/en-US/thunderbird/all/.

I'm not sure if we want the style guide, will discuss.
Flags: needinfo?(sancus)
:sancus, in the past month alone, more than 200,000 sessions visited a releasenotes page on www.mozilla.org from a "direct (none)" source, including tens of thousands hitting the 59.0beta notes. These sessions very likely originate in-product. The redirect strategy proposed in comment 3 will drop all of these visitors at the root of thunderbird.net, which is not an optimal experience for someone who clicked on a "what's new" link in-product. 

Even if we change all in-product links in future releases to point at thunderbird.net, all the previously downloaded versions still have those old links in them. There are at least 12 in-product links that we should evaluate[0]. For example, even if we keep www.mozilla.org/privacy/thunderbird as comment 5 suggests, we still have many in-product links to www.mozilla.org/thunderbird/legal/privacy/, which might redirect thunderbird.net after this change, depending on redirect precedence.

1) Is it your expectation that in-product links to www.mozilla.org/thunderbird/* going forward will point at thunderbird.net?
 
2) Is it OK to drop all Thunderbird users who click an in-product link at the thunderbird.net root going forward?

If no to #2, we need a more granular redirect plan.

---

[0] https://dxr.mozilla.org/comm-central/search?q=regexp%3Awww.mozilla.org%5C%2F((%5C%25LOCALE%5C%25%5C%2F%5C%25APP%5C%25)%7Cthunderbird%7Ccalendar)&redirect=false
Flags: needinfo?(sancus)
The rewrite above doesn't rewrite everything to the root. "mozilla.org/thunderbird/* -> thunderbird.net/*." implies that the request uri is preserved and passed on to thunderbird.net

In the case of release notes, the in-product link is: https://www.mozilla.org/thunderbird/__MOZ_APP_VERSION__/releasenotes/

So, www.mozilla.org/$lang/thunderbird/59.0beta/releasenotes/ should be redirected to www.thunderbird.net/$lang/59.0beta/releasenotes

We should preserve the request uri for rewrites because the paths on thunderbird.net are the same.
Flags: needinfo?(sancus)
> The rewrite above doesn't rewrite everything to the root

Ah, I misunderstood. Thank you for that.
There are several existing URL redirects in bedrock already. I've copied them and what I plan to do with them below: 

Please comment if you have an alternate suggestion :)


>    # bug 1124038
>    redirect(r'^thunderbird/organizations/(?:all-esr\.html|faq/?)$', 'thunderbird.organizations'),

Redirect to https://www.thunderbird.net/organizations/

>    # bug 1124042
>    redirect(r'^thunderbird/features/email_providers\.html$', 'thunderbird.email-providers'),

Delete (I can't find an equivalent the page on thunderbird.net)

>    # bug 1123399, 1150649
>    redirect(r'^thunderbird/all\.html?$', 'thunderbird.latest.all'),
>    redirect(r'^thunderbird/all-beta\.html?$', 'thunderbird.latest.all',
>             to_kwargs={'channel': 'beta'}),
>    redirect(r'^thunderbird/early_releases/downloads/?$', 'thunderbird.latest.all',
>             to_kwargs={'channel': 'beta'}),
>    redirect(r'^thunderbird/early_releases/?$', 'thunderbird.channel'),

Forward to equivalent pages on thunderbird.net

>    # bug 1081917, 1029829, 1029838
>    redirect(r'^thunderbird/releases/(?P<version>0\.\d)\.html$',
>             'http://website-archive.mozilla.org/www.mozilla.org/thunderbird_releasenotes'
>             '/en-US/thunderbird/releases/{version}.html'),
>    # should catch everything 1.* to 29.*
>    redirect(r'^thunderbird/(?P<version>(?:\d|[12]\d)\.[^/]+)/'
>             '(?P<page>firstrun|releasenotes|start|system-requirements|whatsnew)/$',
>             'http://website-archive.mozilla.org/www.mozilla.org/thunderbird_releasenotes'
>             '/en-US/thunderbird/{version}/{page}/'),

Keep.

>    # bug 1209720
>    redirect(r'^thunderbird/releasenotes/?$', 'thunderbird.latest.notes'),

Forward to https://www.thunderbird.net/thunderbird/releases/

>    # bug 1211007
>    redirect(r'^thunderbird/download/?', 'thunderbird.index'),

Forward to https://www.thunderbird.net/

>    # bug 1133266
>    redirect(r'^thunderbird/legal/privacy/?$', 'privacy.notices.thunderbird'),
>    redirect(r'^thunderbird/about/privacy-policy/?$',
>             'privacy.archive.thunderbird-2010-06'),
>    # bug 1196578
>    redirect(r'^thunderbird/about/legal/eula/?$', 'legal.eula'),
>    redirect(r'^thunderbird/about/legal/eula/thunderbird2',
>             'legal.eula.thunderbird-2-eula'),
>    redirect(r'^thunderbird/about/legal/eula/thunderbird',
>             'legal.eula.thunderbird-1.5-eula'),
>    # bug 1204579
>    redirect(r'^thunderbird/2.0.0.0/eula/?$', 'legal.eula.thunderbird-2-eula'),
>    redirect(r'^thunderbird/about/legal/?$', 'legal.terms.mozilla'),
>    redirect(r'^thunderbird/about(/mission)?/?$',
>             'https://wiki.mozilla.org/Thunderbird'),
>    redirect(r'^thunderbird/(about/(careers|contact|get-involved)|community)/?$',
>             'https://wiki.mozilla.org/Thunderbird#Contributing'),
>    redirect(r'^thunderbird/(?P<version>\d\.\d(?:a|b|rc)\d|[6-9]\.0beta)/?$',
>             'http://website-archive.mozilla.org/www.mozilla.org/thunderbird'
>             '/thunderbird/{version}/'),
>    redirect(r'^thunderbird/about/(?P<page>board|press|staff)/',
>             'http://website-archive.mozilla.org/www.mozilla.org/thunderbird'
              '/thunderbird/about/{page}/'),

Keep.

And I'll add our shiny new:

>    redirect(r'^thunderbird/(?P<path>.*)', 'https://www.thunderbird.net/{path}'),
(In reply to Stephanie Hobson [:shobson] from comment #19)
> >    # bug 1124042
> >    redirect(r'^thunderbird/features/email_providers\.html$', 'thunderbird.email-providers'),
> 
> Delete (I can't find an equivalent the page on thunderbird.net)

I forgot this page existed... but it seems to be an orphan? I thought it was linked from the Features page at one point, but I don't see a link on there to it anymore. Grepping the bedrock repo produces no references to it outside the features LESS file, redirects, and the page itself. I don't find it used in the Thunderbird client either.

Other than that everything sounds good.
OK, I've duplicated the calendar and email-providers pages(turns out it IS used in-client for an obscure error message with the URL embedded in a template).

https://www.thunderbird.net/en-US/calendar/
https://www.thunderbird.net/en-US/email-providers/

We also have Bug 1442968 open to provide us independent localization capability on pontoon.mozilla.org.

I would appreciate a scheduled live date for the removal/redirects, just so that I can let everyone know(particularly our release notes publishers, who will need to switch systems) that things are switching over on day X.
:sancus

I'm getting a 404 from the email providers page :(

I found that mozilla.org/thunderbird/system-requirements/ forwards to the most recent system requirements page. Do you want to add a similar redirect to thunderbird.net?

Thank you for moving the calendar page.

Let's schedule a date for the launch after the PR is approved. That way we can be sure it will happen when we say.
Flags: needinfo?(sancus)
404 of email-providers is fixed, I must've forgot to push it live from staging. I also added a rewrite for https://www.thunderbird.net/thunderbird/system-requirements/.

Sounds good for the date scheduling, thanks!
Flags: needinfo?(sancus)
Flags: needinfo?(philipp)
We have an approved and ready to go PR!

So now we need a launch date. How does Tuesday sound? Do you need a specific time?
Flags: needinfo?(sancus)
A specific time isn't needed, as long as it's done during roughly PST daylight hours. Tuesday(April 17) sounds fine to me.
Flags: needinfo?(sancus)
Okay, I've put a reminder in my calendar to push at 10:00am PT. It takes about an hour from button pushing to get into production. (It has to run a gauntlet of tests). I'll update here after it's done.
Commit pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/6858eb808894d6d5ee753689ca1bd46d10f89bb2
Bug 1388914 - Delete thunderbird content

- Removed thunderbird app
  - Removed image, CSS, and JS assets for thunderbird app
  - Remove associated tests
- Remove /projects/calendar and /projects/calendar/holidays
  - Remove associated image, CSS, and JS assets
- Remove release notes
  - Remove associated tests
- Moved redirects from thunderbird app to bedrock redirects
  - Updated to point to thunderbird.net where appropriate
- Added redirects to thunderbird.net
Commit pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/4455862dcf2fcf6b462a8ce88c54b6976f3c1125
Bug 1388914: Redirect /thunderbird to thunderbird.net

Catch the case where there is no slash
It's pushed. Thank you to the Thunderbird team for working closely and being responsive throughout this process :)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Product: www.mozilla.org → www.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: