generated from DFE-Digital/govuk-dotnet-boilerplate
-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix: Prevent Search configuration from failing the application health check #359
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 2 New issues |
🎉 This PR is included in version 2.0.0-next.11 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
killij
added a commit
that referenced
this pull request
Jan 10, 2024
commit 5303806 Author: killij <[email protected]> Date: Wed Jan 10 11:03:12 2024 +0000 fix: Fix CTA column stacking for mobile view (#360) commit 844fe09 Author: killij <[email protected]> Date: Wed Jan 10 10:47:20 2024 +0000 fix: Prevent Search configuration from failing the application health check (#359) commit 770b68f Author: killij <[email protected]> Date: Wed Jan 10 09:38:14 2024 +0000 feat: Replace the existing Resources search with Azure AI Search (#356) * Fix the config key * Initial SearchController * Add new package * Update CSS for search * Search View (wip) * Update the clear filters function * Update filter categories * Order by js to submit form * Css changes * CSS update * Search refactor * Fix the application configuration tests * Add SearchResultsVMFactoryTests * Fix tests * Create SearchResourcesControllerTests.cs * Remove old resources search and substitute new one in * Add resource search facet numbers to filter control * Make resource search filters inclusive OR instead of exclusive AND * Change the gds filter checkbox size to match the design * Update CSS * Display resource properties in search results and resource page * Small refactor * Update SearchResources.cshtml * Remove unused code * Change the query string parameter names and logically OR tags instead of AND * Tidy up old Resource search code * Sonar suggestions * Remove special characters from the search term * Update SearchServiceTests.cs * Set a max query size * Delete search-sort.js * Handle null query * Update SearchResultsVMFactory.cs * Update README.md * Update docker-compose.yml * Update Resource.cshtml commit 671a422 Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed Jan 10 08:40:29 2024 +0000 chore: Bump axios and contentful-export in /Contentful-Schema (#330) Bumps [axios](https://github.com/axios/axios) and [contentful-export](https://github.com/contentful/contentful-export). These dependencies needed to be updated together. Updates `axios` from 0.27.2 to 1.6.1 - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v0.27.2...v1.6.1) Updates `contentful-export` from 7.19.91 to 7.19.95 - [Release notes](https://github.com/contentful/contentful-export/releases) - [Commits](contentful/contentful-export@v7.19.91...v7.19.95) --- updated-dependencies: - dependency-name: axios dependency-type: indirect - dependency-name: contentful-export dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: killij <[email protected]> commit 1575d12 Merge: e055485 6569adf Author: Al West <[email protected]> Date: Wed Dec 20 10:17:25 2023 +0000 Merge pull request #353 from DFE-Digital/tf-ai-search Add Terraform for AI Search Service commit 6569adf Author: Al West <[email protected]> Date: Tue Dec 19 18:01:20 2023 +0000 Add private endpoint for non-free AI Search plans commit c27fd91 Author: Al West <[email protected]> Date: Mon Dec 18 14:10:49 2023 +0000 Add var for search sku commit 361914d Author: Al West <[email protected]> Date: Mon Dec 18 13:54:11 2023 +0000 Update for Terraform Format commit e38be0a Author: Al West <[email protected]> Date: Mon Dec 18 13:52:41 2023 +0000 Add Terraform for AI Search Service commit e055485 Author: killij <[email protected]> Date: Mon Dec 18 11:03:39 2023 +0000 ci: Reintroduce the dotnet-cover config to exclude files marked with ExcludeFromCodeCoverageAttribute (#352) commit c0ce33e Author: killij <[email protected]> Date: Mon Dec 18 09:32:00 2023 +0000 chore: Add the search service (#351) commit f99727f Merge: ca76293 b7b70f1 Author: MartinBelton-gov <[email protected]> Date: Mon Dec 18 08:53:31 2023 +0000 tests: Added Addition UI Tests Merge pull request #350 from DFE-Digital/mab/updateAreasOfPractiseUITests commit b7b70f1 Author: Martin-Belton-gov <[email protected]> Date: Fri Dec 15 12:59:58 2023 +0000 Added Addition UI Tests commit ca76293 Author: killij <[email protected]> Date: Thu Dec 14 09:06:16 2023 +0000 chore: Fix the regression tests (#349) commit 8fb7049 Merge: 68347a1 1ecb68d Author: Al West <[email protected]> Date: Fri Dec 8 12:34:01 2023 +0000 Merge pull request #348 from DFE-Digital/load-test-use-prod-contentful Use Production Contentful environment for Load-Test runs commit 1ecb68d Author: Al West <[email protected]> Date: Fri Dec 8 12:28:53 2023 +0000 Use Production Contentful environment for Load-Test runs Single change to the variables.tf to change Load-Test to use Contentful Prod environment.
🎉 This PR is included in version 2.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.