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.

Sign up for GitHub

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

Jump to bottom

4.2 Segments not sortable by headers #10925

Closed
1 task done
healthstatus opened this issue Mar 2, 2022 · 3 comments · Fixed by #10938
Closed
1 task done

4.2 Segments not sortable by headers #10925

healthstatus opened this issue Mar 2, 2022 · 3 comments · Fixed by #10938
Labels
bug Issues or PR's relating to bugs regression A bug that broke something in the last release segments Anything related to segments

Comments

@healthstatus
Copy link

Mautic Version

4.2.x series

PHP version

7.4.27

What browsers are you seeing the problem on?

Chrome

What happened?

On the segments page, the system is defaulting to sorting by Modified date, you can click the other headers, but sorting on that header does not happen.

How can we reproduce this issue?

Step 1: Go to Segments page
Step 2: click on the column headers to sort by that column

Relevant log output

No response

Code of Conduct

  • I confirm that I have read and agree to follow this project's Code of Conduct
@healthstatus healthstatus added bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged labels Mar 2, 2022
@RCheesley
Copy link
Sponsor Member

Confirmed this bug - @alfredoct96 can you take a look as I think it was your PR that introduced the new columns and the sort order change. Thanks!

@RCheesley RCheesley added segments Anything related to segments regression A bug that broke something in the last release and removed needs-triage For new issues/PRs that need to be triaged labels Mar 4, 2022
@alfredoct96 alfredoct96 mentioned this issue Mar 7, 2022
Merged
@alfredoct96
Copy link
Contributor

Confirmed this bug - @alfredoct96 can you take a look as I think it was your PR that introduced the new columns and the sort order change. Thanks!

I fixed it on this PR #10938

@github-actions
Copy link
Contributor

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If this issue is continuing with the lastest stable version of Mautic, please open a new issue that references this one.

@alfredoct96 alfredoct96 mentioned this issue Mar 16, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Issues or PR's relating to bugs regression A bug that broke something in the last release segments Anything related to segments
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix segments list order CTMobi/mautic
3 participants
@healthstatus @RCheesley @alfredoct96