-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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(locales): use correct lang #283
Conversation
Let's wait for the config to be merged on Vue Router next and make sure it works |
I same issues vue-i18n-next docs. |
Tested with Vue router next, Pinia, and Vite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです!
sorry, my mistake. NVM
// pass a custom lang facetFilter to allow multiple language search | ||
// https://github.com/algolia/docsearch-configs/pull/3942 | ||
facetFilters: facetFilters.concat( | ||
(userOptions.searchParameters || {}).facetFilters || [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about userOptions.searchParameters?.facetFilters || []
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have used it but the syntax is not transpiled and has very recent support only
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not transpiled? I thought it would be 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, it's actually transpiled when building with vitepress! Not when building vitepress itself. It should be fine then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 👍 👍 👍 👍
No description provided.