-
Notifications
You must be signed in to change notification settings - Fork 12
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
Implement Application Translation using next-intl #13
base: master
Are you sure you want to change the base?
Conversation
Rollback Files For Sweep
|
Apply Sweep Rules to your PR?
|
Important Review SkippedReviews are disabled for bot users. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the TipsChat with CodeRabbit Bot (
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
src/i18n/messages/en.json
Outdated
"heroFooter": "No credit card required." | ||
}, | ||
"technologies": { | ||
// Add the keys and translations for the technologies component here |
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.
Sweep: Please fill all required translation strings here
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.
🚀 Wrote Changes
Done.
src/i18n/messages/en.json
Outdated
// Add the rest of the keys and translations for the testimonialHighlight component here | ||
}, | ||
"featureSections": { | ||
// Add the keys and translations for the featureSections component here |
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.
Sweep: Please fill all required translation strings here
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.
🚀 Wrote Changes
Done.
src/components/landing/hero/hero.tsx
Outdated
@@ -1,4 +1,7 @@ | |||
"use client" | |||
import { useTranslations } from "next-intl" | |||
|
|||
const { t } = useTranslations('hero') |
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.
Sweep: This should be added inside the component not in the import statements.
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.
🚀 Wrote Changes
Done.
❌ Could not find files to change Please join our Discord to report this issue. |
3 similar comments
❌ Could not find files to change Please join our Discord to report this issue. |
❌ Could not find files to change Please join our Discord to report this issue. |
❌ Could not find files to change Please join our Discord to report this issue. |
Description
This PR implements application translation using the
next-intl
package in a Next.js TypeScript application. The following components have been translated:src/components/landing/hero/hero.tsx
src/components/landing/partners/technologies.tsx
src/components/landing/testimonials/testimonial-highlight.tsx
src/components/landing/features/column/feature-sections.tsx
The translations have been added to the
src/i18n/messages/en.json
file. The translations are organized into namespaces for each component.Summary of Changes
useTranslations
hook fromnext-intl
in the relevant components.useTranslations
hook to get translations for the appropriate namespaces.src/i18n/messages/en.json
file.Please review the changes and let me know if any further modifications are required.
Fixes #12.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can: