Skip to content

Add searchable to Citation - #9418

Open
gbp wants to merge 3 commits into
developfrom
pg-search-admin-citation-search
Open

Add searchable to Citation#9418
gbp wants to merge 3 commits into
developfrom
pg-search-admin-citation-search

Conversation

@gbp

@gbp gbp commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Relevant issue(s)

Alterative to #9412

What does this do?

Add searchable to Citation

Declare Citation searchable on its source_url, allowing instances to be
searched through the new SearchDocument and the new interface.

gbp added 3 commits July 29, 2026 09:11
Declare Citation searchable on its source_url, allowing instances to be
searched through the new SearchDocument and the new interface.
The admin citation listing searched source URLs with a bespoke SQL LIKE
class method.

Move it onto the shared search interface instead by having the admin
controller call search_scope, forcing the PostgreSQL backend with exact
mode so partial URL fragments still match.

Drop the now-unused Citation.search. The model spec for it goes with it;
the admin controller spec indexes its records and exercises the new path.
@garethrees

Copy link
Copy Markdown
Member

Shall we merge 52869ae on to #9412 to reduce these down to 1 PR?

Don't think it's worth updating the changelog model per model as it's going to get quite big.

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.

2 participants