Skip to content

feat(GAT-8796): allows deferred search providers. allows overriding of provider to search#1663

Merged
loki-sinclair-hdruk merged 4 commits into
devfrom
feat/GAT-8796-be
Jun 15, 2026
Merged

feat(GAT-8796): allows deferred search providers. allows overriding of provider to search#1663
loki-sinclair-hdruk merged 4 commits into
devfrom
feat/GAT-8796-be

Conversation

@loki-sinclair-hdruk

Copy link
Copy Markdown
Contributor

Screenshots (if relevant)

Describe your changes

Default is all with HDR being the immediate, and others being deferred. Also tinkers with queue priorities to better handle clashes of jobs

Search Aggregation now offers:

  • Default - All known providers will run the same criteria for search.
  • Allows overriding of provider to search on. Passing providers array of what providers to call. HDRUK will always run instantly. External providers will be run as deferred jobs. Polling of endpoint with the cache token provided will allow client to render results when ready.

Issue ticket link

Environment / Configuration changes (if applicable)

SEARCH_DEFERRED_TTL - allows the overriding of the default (60 second) cache for deferred search jobs

Requires migrations being run?

If not using the pre-push hook. Confirm tests pass:

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added appropriate unit tests
  • I have created mocks for unit tests (where appropriate)
  • I have added appropriate Behat tests to confirm AC (if applicable)
  • I have added Swagger annotations for new endpoints (if applicable)
  • I have added audit logs for new operation logic (if applicable)
  • I have added new environment variables to the .env.example file (if applicable)
  • I have added new environment variables to terraform repository (if applicable)

…f provider to search - default is all with HDR being the immediate, and others being deferred. Also tinkers with queue priorities to better handle clashes of jobs
@gh-actions-pipelines-app

Copy link
Copy Markdown

🎉 Great job! Your PR title follows the correct format. 🚀

Comment thread app/Http/Controllers/Api/V2/SearchController.php
Comment thread app/Jobs/DeferredProviderSearch.php
@loki-sinclair-hdruk loki-sinclair-hdruk merged commit 5045ee0 into dev Jun 15, 2026
2 checks passed
@loki-sinclair-hdruk loki-sinclair-hdruk deleted the feat/GAT-8796-be branch June 15, 2026 08:33
gh-actions-pipelines-app Bot pushed a commit that referenced this pull request Jun 25, 2026
## [2.43.0](v2.42.0...v2.43.0) (2026-06-25)

### ✨ Features

* **GAT-0000:** add CRUK project grants with dataset linkage (#1656) ([8426b9f](8426b9f)), closes [GAT-0000](undefinedGAT-0000)
* **GAT-0000:** seed 20 CRUK datasets from cruk_dummy_data fixtures (#1666) ([0f738f5](0f738f5)), closes [GAT-0000](undefinedGAT-0000)
* **GAT-8501:** Make TeamAdmin an actual admin (#1674) ([6eba7a1](6eba7a1)), closes [GAT-8501](undefinedGAT-8501)
* **GAT-8796:** Add dataSource to filter seeder (#1662) ([ea3b63e](ea3b63e)), closes [GAT-8796](undefinedGAT-8796)
* **GAT-8796:** allows deferred search providers. allows overriding of provider to search (#1663) ([5045ee0](5045ee0)), closes [GAT-8796](undefinedGAT-8796)
* **GAT-8895:** Add email into workflow when Integration job fails (#1653) ([988b847](988b847)), closes [GAT-8895](undefinedGAT-8895)
* **GAT-8977:** Investigate logging when the widget starts running on third-party websites (#1654) ([d5d1fec](d5d1fec)), closes [GAT-8977](undefinedGAT-8977) [Investigate](undefinedgate)
* **GAT-8978:** brings search aggregator more inline with production than PoC. (#1657) ([db273d4](db273d4)), closes [GAT-8978](undefinedGAT-8978) [aggregator](undefinedgator)
* **GAT-9012:** 3 - tweaks redis config and job firing (#1669) ([8034869](8034869)), closes [GAT-9012](undefinedGAT-9012)
* **GAT-9012:** First round of template work - refactor the seeder fi… (#1667) ([da978e0](da978e0)), closes [GAT-9012](undefinedGAT-9012)
* **GAT-9012:** Fixes a small edge case where some integration errors… (#1668) ([9a526ec](9a526ec))
* **GAT-9027:** Do not send success email when no changes have been r… (#1671) ([db0fa09](db0fa09)), closes [GAT-9027](undefinedGAT-9027)

### 🐛 Bug Fixes

* **GAT-0000:** add x-partner-context header (#1673) ([5d298d9](5d298d9)), closes [GAT-0000](undefinedGAT-0000)
* **GAT-8796:** Fixes sorting missing from refactor. Fixes naming of n… (#1664) ([c481cac](c481cac))
* **GAT-8812:** updated semantic pipeline (#1655) ([c0215dc](c0215dc)), closes [GAT-8812](undefinedGAT-8812)
* **GAT-8872:** Content Variance SQL Injection: Database Server (#1651) ([572b06c](572b06c)), closes [GAT-8872](undefinedGAT-8872)
* **GAT-8925:** Properly handles SDE expiry isolated from general cohort expiry (#1658) ([def0e82](def0e82)), closes [GAT-8925](undefinedGAT-8925)
* **GAT-8926:** fix collection has datasets (#1652) ([74311ac](74311ac))
* **GAT-9031:** Ensures we don\'t leak any sensitive infrastructure in… (#1672) ([0aad073](0aad073)), closes [GAT-9031](undefinedGAT-9031)
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