1261378 - "View Pocket List" requires sign-in when opened in Private Browsing, although pocket saving works fine

Status: REOPENED

(Firefox :: Pocket, defect, P2)

Description

[Paul Silaghi, QA [:pauly]]

Attachment: view pocket list private browsing.png

[Affected versions]:
- 46b6

[Affected platforms]:
- all

[Steps to reproduce]:
1. Start FX with a new profile
2. Sign in to Pocket
3. Open a new private window
4. Open a page and save it to Pocket
5. Click on the "View Pocket List" button

[Expected result]:
- "View Pocket List" shouldn't ask for credentials

[Actual result]:
- "View Pocket List" requires sign-in when opened in Private Browsing, although pocket saving works fine

[Regression range]:
- reproduced on FX 45, 48.0a1 (2016-03-31)

Comment 1

[:Gijs (he/him)]

Again, if this applies to 45, why is this blocking bug 1215694 ?

Comment 2

[Shane Caraveo (:mixedpuppy)]

I'm not certain this is actually a bug.  If anything, it is a bug that the pocket addon credentials are used in the private window.

Comment 3

[Paul Silaghi, QA [:pauly]]

Pocked moved to a system add-on in bug 1215694, which landed on FX 46. So, I though all the new bugs should block bug 1215694. The 45 flag is set so people to know this is not a recent regression caused by the Pocket moving.

Comment 4

[Jessilyn Davis]

This bug has been closed due to inactivity and/or the potential for this bug to no longer be an issue with the new Discovery Stream-powered New Tab experience, and improvements to Pocket’s integration with Firefox.

Please help us triage by reopening if this issue still persists and should be addressed.

Thanks!

Comment 5

[:Gijs (he/him)]

The behaviour in comment #0 still occurs; as comment #2 suggests, it's not clear which of the two cases (saving to pocket or showing the list) needs fixing here - but we should make a decision, the status quo is unexpected.

Comment 6

[Jessilyn Davis]

Thanks for reopening :Gijs!

Comment 7

[Jessilyn Davis]

NI'ing Product & UX for advice on how this should work.

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

Redirect needinfos that are pending on inactive users to the triage owner.
:thecount, since the bug has high priority, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 9

[:Gijs (he/him)]

We removed the "view pocket list" item.

Comment 10

[Scott [:thecount] Downe]

We have a "View My List" button in the Pocket panels which is essentially the same, no?

When I click it while in private browsing, it seems to save just fine redirects me to a non private tab. That feels broken to me.

Comment 11

[:Gijs (he/him)]

(In reply to Scott [:thecount] Downe from comment #10)

We have a "View My List" button in the Pocket panels which is essentially the same, no?

When I click it while in private browsing, it seems to save just fine redirects me to a non private tab. That feels broken to me.

You are right, sorry for missing that. I'll leave you the needinfo then! ;-)