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

fix(locales): use correct lang #283

Merged
merged 9 commits into from
Apr 17, 2021
Merged

fix(locales): use correct lang #283

merged 9 commits into from
Apr 17, 2021

Conversation

posva
Copy link
Member

@posva posva commented Apr 8, 2021

No description provided.

@posva posva marked this pull request as draft April 8, 2021 15:44
@posva
Copy link
Member Author

posva commented Apr 8, 2021

Let's wait for the config to be merged on Vue Router next and make sure it works

@kazupon
Copy link
Member

kazupon commented Apr 14, 2021

I same issues vue-i18n-next docs.
I sent PR docsearch-configs (ref: algolia/docsearch-configs#3957 (review))

@posva posva marked this pull request as ready for review April 15, 2021 11:43
@posva
Copy link
Member Author

posva commented Apr 15, 2021

Tested with Vue router next, Pinia, and Vite

Copy link
Member

@kazupon kazupon left a 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 || []
Copy link
Member

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 || [] ?

Copy link
Member Author

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

Copy link
Member

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 🤔

Copy link
Member Author

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

Copy link
Member

@kiaking kiaking left a comment

Choose a reason for hiding this comment

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

👍 👍 👍 👍 👍

@kiaking kiaking merged commit de89c1e into master Apr 17, 2021
@kiaking kiaking deleted the fix/correct-lang-locales branch April 17, 2021 12:43
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants