Skip to content
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

Upgrade docsearch to v3 #5129

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Conversation

pnbrown
Copy link

@pnbrown pnbrown commented Feb 28, 2023

Updating docsearch to fix the search functionality on the site

@pnbrown pnbrown requested a review from a team as a code owner February 28, 2023 18:41
@pnbrown pnbrown requested review from tsaarni and sunjayBhatia and removed request for a team February 28, 2023 18:41
@pnbrown pnbrown marked this pull request as draft February 28, 2023 18:44
@pnbrown
Copy link
Author

pnbrown commented Feb 28, 2023

Closes #5086

@codecov
Copy link

codecov bot commented Feb 28, 2023

Codecov Report

Merging #5129 (dd4dbe8) into main (64fef09) will increase coverage by 0.04%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5129      +/-   ##
==========================================
+ Coverage   77.97%   78.02%   +0.04%     
==========================================
  Files         137      138       +1     
  Lines       17237    17517     +280     
==========================================
+ Hits        13441    13668     +227     
- Misses       3536     3583      +47     
- Partials      260      266       +6     
Impacted Files Coverage Δ
internal/status/routeconditions.go 51.04% <0.00%> (-6.61%) ⬇️
internal/dag/gatewayapi_processor.go 95.49% <0.00%> (-0.33%) ⬇️
cmd/contour/serve.go 22.20% <0.00%> (-0.10%) ⬇️
internal/controller/grpcroute.go 25.80% <0.00%> (ø)
internal/dag/policy.go 94.76% <0.00%> (+0.01%) ⬆️
internal/contour/handler.go 90.00% <0.00%> (+0.07%) ⬆️
internal/dag/cache.go 94.36% <0.00%> (+0.10%) ⬆️
internal/k8s/kind.go 60.65% <0.00%> (+1.47%) ⬆️
internal/gatewayapi/helpers.go 87.94% <0.00%> (+2.98%) ⬆️

@pnbrown pnbrown added area/website Issues or PRs related to the web site. release-note/docs A documentation change for the release notes. labels Feb 28, 2023
@pnbrown pnbrown marked this pull request as ready for review February 28, 2023 18:49
@sunjayBhatia sunjayBhatia added release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes. release-note/docs A documentation change for the release notes. and removed release-note/docs A documentation change for the release notes. release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes. labels Feb 28, 2023
@sunjayBhatia
Copy link
Member

docs release note guidelines here: https://github.com/projectcontour/contour/blob/main/changelogs/unreleased/docs-sample.md

something simple like "Upgrade algolia docsearch to v3" should be good

@sunjayBhatia
Copy link
Member

looks like it is working on the deploy preview: https://deploy-preview-5129--projectcontour.netlify.app/

@skriss
Copy link
Member

skriss commented Feb 28, 2023

looks like it is working on the deploy preview: https://deploy-preview-5129--projectcontour.netlify.app/

Yep, preview looks great to me, the updated UI seems pretty nice!

Updating docsearch to fix the search functionality on the site

Signed-off-by: Nigel Brown <nigelb@vmware.com>
Copy link
Member

@skriss skriss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sunjayBhatia sunjayBhatia merged commit c0d8219 into projectcontour:main Feb 28, 2023
@pnbrown pnbrown deleted the update-search branch February 28, 2023 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/website Issues or PRs related to the web site. release-note/docs A documentation change for the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants