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

The wrong segment may be selected in the Segment editor #11787

Closed
mattab opened this issue Jun 13, 2017 · 0 comments · Fixed by #11790
Closed

The wrong segment may be selected in the Segment editor #11787

mattab opened this issue Jun 13, 2017 · 0 comments · Fixed by #11790
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Jun 13, 2017

After creating a new segment, the reports try hard to apply this new segment instead of the last selected segment.

How to reproduce:

  • Go to Visitor Log (or any report)
  • Click Add new segment (any segment will do) called Test segment
  • Click Save & Apply
  • Note the URL query string (before hash tag) includes the &segment= parameter
  • The visitor log displays the Test segment visits (as expected)
  • Click on All Visits in segment editor
  • The visitor log displays all visits (as expected)
  • Then click on any other menu link, such as Overview.
  • Got: the report applies segment Test segment
  • Expected: the report should apply the previously clicked segment ie. All Visits

(Seems the problem is that we set the &segment= parameter in the URL before hash tag and it gets read and set after the hash tag. Wondering why we set this &segment= in the query string in the first place)

@mattab mattab added Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should. labels Jun 13, 2017
@mattab mattab added this to the 3.0.5 milestone Jun 13, 2017
@sgiehl sgiehl self-assigned this Jun 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. Regression Indicates a feature used to work in a certain way but it no longer does even though it should.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants