Skip to content

fix: failed to update search index after switching lang#1950

Merged
SoonIter merged 4 commits intomainfrom
switch_lang_0312
Mar 12, 2025
Merged

fix: failed to update search index after switching lang#1950
SoonIter merged 4 commits intomainfrom
switch_lang_0312

Conversation

@chenjiahan
Copy link
Copy Markdown
Member

Summary

  1. Removes the if (initStatus === 'initial') condition, because the initStatus will always be initial if the user does not click on the search bar.
  2. Call searcher.fetchSearchIndex() after switching languages to prefetch index files.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 12, 2025

Deploy Preview for aquamarine-blini-95325f ready!

Name Link
🔨 Latest commit 96588a0
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-blini-95325f/deploys/67d13cce44fff9000835c000
😎 Deploy Preview https://deploy-preview-1950--aquamarine-blini-95325f.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 80 (🔴 down 14 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 12, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit 96588a0
🔍 Latest deploy log https://app.netlify.com/sites/rspress-v2/deploys/67d13cced93c990008e868d4
😎 Deploy Preview https://deploy-preview-1950--rspress-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 12, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit cef37ce
🔍 Latest deploy log https://app.netlify.com/sites/rspress-v2/deploys/67d13e968d5daf000813e811
😎 Deploy Preview https://deploy-preview-1950--rspress-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 12, 2025

Deploy Preview for aquamarine-blini-95325f ready!

Name Link
🔨 Latest commit cef37ce
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-blini-95325f/deploys/67d13e960fbbb20008656cd6
😎 Deploy Preview https://deploy-preview-1950--aquamarine-blini-95325f.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 91 (🔴 down 3 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@chenjiahan chenjiahan requested a review from SoonIter March 12, 2025 07:54
@SoonIter SoonIter merged commit a2fec82 into main Mar 12, 2025
11 checks passed
@SoonIter SoonIter deleted the switch_lang_0312 branch March 12, 2025 09:05
@chenjiahan chenjiahan mentioned this pull request Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants