Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Unable to add site to Pocket #9669

Closed
LaurenWags opened this issue Jun 22, 2017 · 10 comments
Closed

Unable to add site to Pocket #9669

LaurenWags opened this issue Jun 22, 2017 · 10 comments

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Jun 22, 2017

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Unable to add a site to Pocket. Whenever I click on the pocket icon, I get a login page that spins.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    MacOS (asked @srirambv to confirm on Windows as well)

  • Brave Version (revision SHA):
    Name Version
    Brave 0.17.6
    rev 30a3eee
    Muon 4.1.2

  • Steps to reproduce:

  1. Start with clean install of 0.17.6
  2. Enable Pocket in Preferences > Extensions (Pocket Version is 2.1.32)
  3. Login to Pocket on tab that opens.
  4. You are logged in and navigated to your list.
  5. Open a new tab and navigate to a site (I used nytimes.com)
  6. Click on a link to open a news article
  7. Select the Pocket icon.
  8. Pocket login page opens in a new tab and just spins. Site is not added to your list. See screenshot below.

cc: @jonathansampson @kevinlawler

@LaurenWags LaurenWags added the bug label Jun 22, 2017
@LaurenWags LaurenWags added this to the 0.17.x (Beta Channel) milestone Jun 22, 2017
@srirambv
Copy link
Collaborator

Confirmed on Windows as well.

@bsclifton
Copy link
Member

Is this the same issue as #9117? ex: if you exit Brave and relaunch, does the issue still happen? (or is it only on the first load)

@bsclifton
Copy link
Member

setting to same milestone as #9117

@bsclifton bsclifton modified the milestones: 0.19.x (Nightly Channel), 0.17.x (Beta Channel) Jun 23, 2017
@LaurenWags
Copy link
Member Author

@bsclifton if I close and relaunch Brave I am still not able to add a site to Pocket (repeating steps 5-8 after Browser is relaunched).

@bsclifton
Copy link
Member

@jonathansampson would you be able to check this out? If Pocket isn't working on 0.17.x, should we block the release?

cc: @bbondy @bradleyrichter @alexwykoff

@LaurenWags LaurenWags modified the milestones: 0.17.x (Beta Channel), 0.19.x (Nightly Channel) Jun 27, 2017
@srirambv
Copy link
Collaborator

Works on 0.17.13. Able to add site in pocket when logged in. #9117 still seems to be an issue but pocket functionality seems to be back

@alexwykoff alexwykoff modified the milestones: 0.18.x (Developer Channel), 0.17.x (Beta Channel) Jun 30, 2017
@alexwykoff
Copy link
Contributor

Appears to be working, though the first time interaction needs work.

@bbondy
Copy link
Member

bbondy commented Jul 11, 2017

@jonathansampson is going to verify if this is only for existing tabs and not for new tabs once enabled.

@jonathansampson
Copy link
Collaborator

jonathansampson commented Jul 25, 2017

I think this might be a dupe of #4687. I followed the steps, and found the extension_login_success page remaining open. That being said, the site was added to my list.

The login page was only shown for the first engagement of the browserAction icon. All subsequent engagements resulted in no new tab (only the item being added to my list).

@jonathansampson
Copy link
Collaborator

This issue is a confirmed dupe of #4687.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.