Skip to content

docs: update 02.usage.md#3396

Merged
BobbieGoede merged 1 commit intonuxt-modules:mainfrom
AnzhiZhang:patch-1
Mar 8, 2025
Merged

docs: update 02.usage.md#3396
BobbieGoede merged 1 commit intonuxt-modules:mainfrom
AnzhiZhang:patch-1

Conversation

@AnzhiZhang
Copy link
Copy Markdown
Contributor

🔗 Linked issue

#3395

❓ Type of change

  • 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This resolves ambiguous i18n.config.ts path in the documentation usage page.

resolve #3395

📝 Checklist

  • I have linked an issue or discussion.
  • I have added tests (if possible).
  • I have updated the documentation accordingly.

@AnzhiZhang
Copy link
Copy Markdown
Contributor Author

It would be better to update the error message in layers.ts as well. Please let me know if you want me to change it.

@AnzhiZhang AnzhiZhang changed the title Update 02.usage.md docs: update 02.usage.md Mar 8, 2025
@basilky
Copy link
Copy Markdown

basilky commented Mar 8, 2025

I was wondering why the configuration in the project root is not getting read. The file path shown in error is existing in the project root

@AnzhiZhang
Copy link
Copy Markdown
Contributor Author

I was wondering why the configuration in the project root is not getting read. The file path shown in error is existing in the project root

Beacuse it is reading the i18nDir, see details in the issue.

Copy link
Copy Markdown
Member

@BobbieGoede BobbieGoede left a comment

Choose a reason for hiding this comment

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

Thank you for contributing! 🙏

@BobbieGoede BobbieGoede merged commit 8e45c34 into nuxt-modules:main Mar 8, 2025
@AnzhiZhang
Copy link
Copy Markdown
Contributor Author

Thank you for contributing! 🙏

Thank you. Is there any plan to update the warning message mentioned above?

@BobbieGoede
Copy link
Copy Markdown
Member

@AnzhiZhang
Just opened #3398, this changes the log to print the actual directory we're checking, instead of the layer's root directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ambiguous i18n.config.ts path

3 participants