Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PLP filters break when opening Sign In popup #468

Closed
franciscomdrito opened this issue Nov 24, 2020 · 1 comment · Fixed by #1286
Closed

PLP filters break when opening Sign In popup #468

franciscomdrito opened this issue Nov 24, 2020 · 1 comment · Fixed by #1286
Assignees
Milestone

Comments

@franciscomdrito
Copy link
Contributor

franciscomdrito commented Nov 24, 2020

Actual Behavior

PLP filters have an inconsistent behavior, when selecting a filter and after, opening the sign in popup, the filters either works, lose the selection or all filters disappear (specially if we refresh the page). The behavior of the product list itself seems to be ok, but filters wont reflect it. Seems a concurrency issue, maybe caused by cancelling the routing when opening the sing in button.

Expected Behavior

The sign in popup shouldn't affect the behavior of the PLP in general

Steps to Reproduce the Bug

Steps to reproduce the behavior:

  1. Go to Navigation and Select Computers
  2. Select Tablets
  3. Select Acer
  4. Select filter Price 1000 >
  5. Click sign in and close popup (sometimes happens the first time, sometimes we need to do more than once, seems a concurrency issue)

Environment Details

  • Desktop: *

    • OS: Windows 10
    • Browser Chrome
    • 0.25
      Additional Context, like Screenshots, Log File Snippets etc.

image

After clicking the sing in button and closing it without logging in:

image

or in sometimes (or when refreshing):

image

AB#80093

@franciscomdrito franciscomdrito added the bug Something isn't working label Nov 24, 2020
@rkarl-ish rkarl-ish added bug:minor and removed bug Something isn't working labels Dec 2, 2020
@Eisie96 Eisie96 self-assigned this May 12, 2021
@jometzner jometzner self-assigned this May 20, 2021
@shauke shauke added this to the 0.32 milestone Jun 4, 2021
@rkarl-ish rkarl-ish removed this from the 1.0 milestone Jul 28, 2021
@shauke
Copy link
Collaborator

shauke commented Feb 16, 2022

Comment from @dhhyi regarding PR #696 (closed and we are open for a better solution):

I also don't think take(1) is a solution here.
I think the problem is the select(selectQueryParams) in determineParams$, as it will start firing when the routing is started. Postponing it unto successful navigation should be the solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants