Skip to content

refactor(search): rebuild feature branch on main via cherry-pick - #1215

Merged
wesleyboar merged 31 commits into
epic/cmd-187from
refactor/rebuild-search-on-main
Jul 27, 2026
Merged

refactor(search): rebuild feature branch on main via cherry-pick#1215
wesleyboar merged 31 commits into
epic/cmd-187from
refactor/rebuild-search-on-main

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Jul 27, 2026

Copy link
Copy Markdown
Member

Overview

Rebuilds the search feature on current main instead of stale release/v4.36.X.

Warning

First run on a fresh database will crash manage.py migrate itself, because apps/search_page/apps.py's ready() hook queries cms_page before it exists. Workaround: set SEARCH_PAGE_AUTO_SETUP = False, migrate, then set it back to True and restart. Details.

Important

Should instead:

Related

Changes

  • rebuilt branch from main + cherry-picked search-only commits
  • dropped unrelated inherited commits (favicons, breadcrumbs, image-zoom, etc.)

Testing / UI

See #1129.

So that it can be loaded from CDN. Use cases are:
- if search template is broken
- for custom search page
- for non-Core website
Compared to "cms-search", "google-search" is:
- more accurate
- more obvious to those not familiar with Core-CMS
So dev sees 'google-search' from CSS before seeing it in HTML `id` attr.
This had create dbreadcrumsb before.
Final markup undecided, so keep it simple, and easy to understand.
Added styles to fix link UI and UX.
Added styles to fix MORE link UI and UX.
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@wesleyboar
wesleyboar changed the base branch from feat/template-and-setting-for-google-search to main July 27, 2026 18:01
@wesleyboar
wesleyboar changed the base branch from main to epic/cmd-187 July 27, 2026 21:13
@wesleyboar
wesleyboar merged commit cd06d92 into epic/cmd-187 Jul 27, 2026
@wesleyboar
wesleyboar deleted the refactor/rebuild-search-on-main branch July 27, 2026 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant