1134286 - Insert Reading List FTU page by default into all user's Reading Lists

Status: RESOLVED FIXED

(Firefox :: Tours, defect)

Description

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

For FF38 we'll be deploying a Reading List First Time Use Tour (FTU).

The current wireframes (bug 1129536) detail entries into the first time use tour which include a Reading List entry.

This bug is requesting the insertion of a URL into the Reading List that will be included with all instances of FF38.

Comment 1

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

:clarkbw -

Note the locale redirection on bedrock goes: m.o/example (301) m.o/en-US/example

Questions:

# Do you need anything other than a URL (without locale) to get this done?
# Will the translated <title> tag be displayed in the user's sidebar?
# When will we have to give you the URL?

Thanks

Comment 2

[Bryan Clark (DevTools PM) [:clarkbw]]

(In reply to Cory Price [:ckprice] from comment #1)
> :clarkbw -
> 
> Note the locale redirection on bedrock goes: m.o/example (301)
> m.o/en-US/example
> 
> Questions:
> 
> # Do you need anything other than a URL (without locale) to get this done?

Just the URL should work.

> # Will the translated <title> tag be displayed in the user's sidebar?

NI: Dolske to help answer these.

My assumption is that Fx will reach out to grab meta data about the page and during that time should get the correct page through the redirect.  Dolske, am I too hopeful?

> # When will we have to give you the URL?

The sooner the better.  Perhaps we can use an allizom link during the Nightly->Aurora cycles and we could uplift the real links later on.

Dolske, what do you think?

bug 1132988 is tracking the SUMO articles that will also be default entries; it might be good to keep an eye on that too.

Comment 3

[Justin Dolske [:Dolske]]

(In reply to Bryan Clark (Firefox PM) [:clarkbw] from comment #2)

> > # Will the translated <title> tag be displayed in the user's sidebar?
> 
> NI: Dolske to help answer these.
> 
> My assumption is that Fx will reach out to grab meta data about the page and
> during that time should get the correct page through the redirect.  Dolske,
> am I too hopeful?

I presume we'll tell Firefox to a URL like https://mozilla.org/%LOCALE/whatever.html, which we'll format with the user's actual locale before actually adding it to the reading list. So when the usual metadata collection kicks in, it would fetch the translated version.


> > # When will we have to give you the URL?
> 
> The sooner the better.  Perhaps we can use an allizom link during the
> Nightly->Aurora cycles and we could uplift the real links later on.
> 
> Dolske, what do you think?

Would probably be good to at least have a placeholder URL in the short term. The actual content shouldn't be critical.

Comment 4

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

We've decided on the URL for the reading list.

It will be:

https://www.mozilla.org/%LOCALE/firefox/reading/start/?utm_source=firefox-browser&utm_medium=firefox-browser

:garethc - will you please review the parameters here? Ref UX bug 1129536

Comment 5

[Gareth Cull [:garethc]]

Looks good to me.

Comment 6

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

URL defined. Closing this bug.

Comment 7

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

Attachment: Final URL

Comment 8

[Alex Gibson [:agibson]]

(In reply to Cory Price [:ckprice] from comment #7)
> Created attachment 8570590 [details]
> Final URL

Note there's a missing `%` here after `%LOCALE`. Should be `%LOCALE%`

Comment 9

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

Attachment: Final URL

Thanks Alex - updated the attachment

Comment 10

[Axel Hecht [:Pike]]

I suggest to drop the %LOCALE% from the URL and to rely on server-side language negotiation.

Comment 11

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

(In reply to Axel Hecht [:Pike] from comment #10)
> I suggest to drop the %LOCALE% from the URL and to rely on server-side
> language negotiation.
We can work with either.

On bedrock

https://www.mozilla.org/firefox/reading/start/

redirects to

https://www.mozilla.org/en-US/firefox/reading/start/

Comment 3 suggested including the %LOCALE% in the URL.

Comment 12

[Bryan Clark (DevTools PM) [:clarkbw]]

dolske, does it matter to you if we drop the %LOCALE% from the url?

Comment 13

[Justin Dolske [:Dolske]]

Nope, I'm happy to do whatever L10N prefers.

I'm a little confused as to which is the "right" way -- seems like we're using a mix.

Looks like 1136570 hasn't yet added the URLs to the default list, but we can add it sans-%LOCALE% when it does.

Comment 14

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

Attachment: Final URL - Mar 11

Updating URL per Alex's request and Dolske's confirmation

Comment 15

[Cory Price [:ckprice] (bugmail disabled, NI me!)]

Attachment: Final URL - Mar 11