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

Spot.im on aol.com #10830

Open
lukemulks opened this issue Sep 6, 2017 · 2 comments
Open

Spot.im on aol.com #10830

lukemulks opened this issue Sep 6, 2017 · 2 comments

Comments

@lukemulks
Copy link
Collaborator

lukemulks commented Sep 6, 2017

Same result on macOS:
screen shot 2017-09-06 at 2 59 22 pm

  • Expected result:
    Spot.im comment features function properly, expanding when needed and allowing for engagement.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes.

  • Is this an issue in the currently released version?
    Yes.

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

Note this is probably something similar to what we encounter with Disqus, when ad features are enabled by the publisher.

We should:

  • Determine if we're legitimately blocking requests that pass user data.
  • If the requests that are blocked are not passing user data, we could make an exception for spot.im
  • Identify the specific request being blocked and test the exception.
@bsclifton bsclifton added this to the 0.20.x (Developer Channel) milestone Sep 6, 2017
@diracdeltas
Copy link
Member

diracdeltas commented Sep 6, 2017

the issue is that spot.im as a third-party cannot access localStorage so it errors when the 'load more' button is clicked. changing to 'allow all cookies' fixes the issue

#1268
#1991

@NejcZdovc NejcZdovc added priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). and removed priority/low (deprecated) labels Sep 19, 2017
@alexwykoff alexwykoff removed this from the 0.20.x (Beta Channel) milestone Oct 24, 2017
@bsclifton bsclifton added this to the Backlog (Prioritized) milestone Nov 22, 2017
@bsclifton bsclifton added workaround/allow-all-cookies and removed priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). site-bug labels Sep 3, 2018
@bsclifton bsclifton modified the milestones: Backlog (Prioritized), Triage Backlog Sep 18, 2018
@pes10k
Copy link
Contributor

pes10k commented Oct 1, 2018

The linked to page in the issue no longer demonstrates the issue in any browser since the chat widget has been removed. So I think the issue should be closed, or at the least, some new site found that demonstrates the problem

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

No branches or pull requests

6 participants