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

nuxt3 integration #1044

Merged
merged 20 commits into from
Jul 8, 2022
Merged

nuxt3 integration #1044

merged 20 commits into from
Jul 8, 2022

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented Jul 5, 2022

No description provided.

@kazupon kazupon added Type: Example Includes example and demo code fixes Type: Documentation A documetation fixes labels Jul 5, 2022
@kazupon kazupon changed the title nuxt3 integration docs nuxt3 integration Jul 5, 2022
docs/.vitepress/config.js Outdated Show resolved Hide resolved
kazupon and others added 8 commits July 5, 2022 22:42
Co-authored-by: Sébastien Chopin <seb@nuxtjs.com>
Co-authored-by: Sébastien Chopin <seb@nuxtjs.com>
Co-authored-by: Sébastien Chopin <seb@nuxtjs.com>
Co-authored-by: Sébastien Chopin <seb@nuxtjs.com>
Co-authored-by: Sébastien Chopin <seb@nuxtjs.com>
Co-authored-by: Sébastien Chopin <seb@nuxtjs.com>
+ <option value="ja">ja</option>
+ </select>
+ </form>
+ </div>
Copy link
Contributor

Choose a reason for hiding this comment

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

</div> is already in app.vue, not a new addition, so no plus sign is needed at the beginning of the line.

legacy: false,
globalInjection: true,
locale: 'en',
- messages: {
Copy link
Contributor

Choose a reason for hiding this comment

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

It doesn't seem like there should be a minus sign in front of messages, since en, fr, ja are all in the messages option.

Co-authored-by: Fuqiao Xue <xfq@w3.org>
kazupon and others added 7 commits July 6, 2022 18:54
Co-authored-by: Fuqiao Xue <xfq@w3.org>
Co-authored-by: Fuqiao Xue <xfq@w3.org>
Co-authored-by: Fuqiao Xue <xfq@w3.org>
Co-authored-by: Fuqiao Xue <xfq@w3.org>
Co-authored-by: Fuqiao Xue <xfq@w3.org>
Co-authored-by: Fuqiao Xue <xfq@w3.org>
@kazupon
Copy link
Member Author

kazupon commented Jul 6, 2022

@xfq
Thank you so much for your review! ❤️

+ alias: {
+ 'vue-i18n': 'vue-i18n/dist/vue-i18n.runtime.esm-bundler.js'
+ }
+ },
Copy link
Member Author

@kazupon kazupon Jul 7, 2022

Choose a reason for hiding this comment

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

@atinux
vite.resolve.alias isn't needed no longer, so It's set in @intlify/unplugin-vue-i18n.
https://github.com/intlify/bundle-tools/releases/tag/unplugin-vue-i18n%400.3.0

@kazupon kazupon merged commit ff8286c into master Jul 8, 2022
@kazupon kazupon deleted the example/nuxt3-integration branch July 8, 2022 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation A documetation fixes Type: Example Includes example and demo code fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants