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

Show Default Site Search index on top #26173

Closed
Tracked by #26171
erickgonzalez opened this issue Sep 19, 2023 · 1 comment · Fixed by #26198
Closed
Tracked by #26171

Show Default Site Search index on top #26173

erickgonzalez opened this issue Sep 19, 2023 · 1 comment · Fixed by #26198

Comments

@erickgonzalez
Copy link
Contributor

erickgonzalez commented Sep 19, 2023

Parent Issue

No response

Task

Always show the default site search index first.

Steps

1- Create 5 sitesearch indexes.
2- Set one as default.
3- Should be move to the top of the list.
4- Set another one as default.
5- Now that one should be at the top.

Proposed Objective

Customer Support

Proposed Priority

Priority 2 - Important

Acceptance Criteria

The default Site Search index must be on top of the list. In the case of pagination, it must be on the first page.

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

AndreyDotcms added a commit that referenced this issue Sep 20, 2023
AndreyDotcms added a commit that referenced this issue Sep 20, 2023
AndreyDotcms added a commit that referenced this issue Sep 25, 2023
github-merge-queue bot pushed a commit that referenced this issue Sep 25, 2023
* #26173 adding fix and test

* #26173 remove unnecessarily import

* #26173 renaming and validating

---------

Co-authored-by: erickgonzalez <erick.gonzalez@dotcms.com>
@erickgonzalez erickgonzalez reopened this Sep 25, 2023
@bryanboza
Copy link
Member

Fixed, now we are showing the default one on the top of the list

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants