Skip to content

Show search API failures in document results - #1441

Closed
the8thbit wants to merge 1 commit into
MuckRock:mainfrom
the8thbit:fix/solr-error-1438
Closed

the8thbit wants to merge 1 commit into
MuckRock:mainfrom
the8thbit:fix/solr-error-1438

Conversation

@the8thbit

Copy link
Copy Markdown

Search failures now show the API error in the results list and clear stale results when a new query fails.

Closes #1438.

  • Behaviour changes: Search errors remain visible instead of appearing as an empty successful search; a subsequent successful query clears the error. Existing results remain available if pagination fails.
  • Migrations: None.
  • What to check:
    • Search state failure/recovery and escaped error rendering; full unit suite (1184 passed, 9 todo).
    • PUBLIC_ENV=development npm run check (0 errors, 0 warnings).
    • PUBLIC_ENV=development npm run build.
    • Prettier and git diff --check.

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.

Surface Solr errors in the frontend

2 participants