-
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
Feature/i18n #814
Feature/i18n #814
Conversation
@lutangar @JalilArfaoui @christpet The extension is ready to be translated, if you want to check. Otherwise I have a little problem with semaphore which hasn't validated one of my commit |
src/locales/resources.ts
Outdated
import fr from './fr'; | ||
import en from './en'; | ||
|
||
export default { |
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.
<3
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.
❤️
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.
Looks great!
Don't bother about this, ... happens sometimes. You kept the WIP flag on this, do you plan to keep working on this branch? Shall we merge it? |
Sorry i forgot to remove the flag, yes we can merge |
@lutangar it's done for me, do you want to merge ? |
🎉 This PR is included in version 3.64.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
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.
Sorry for late feedback …
♥
Is there any specific reason you sometimes used the HOC, and sometimes the hook ?
@JalilArfaoui I can't use Hook on class components so I use HOC instead |
@gregoirelacoste oh yes of course … I knew I was missing something obvious 😅 Thanks ! |
@gregoirelacoste just noticed this slight bug in the extension on the Firefox Staging, latest version 3.67.2: Bug description:
How to reproduce:
|
I have another one here @gregoirelacoste: As you can see, it says "Demander une information ici" instead of the English translation This is on the Firefox staging version See also "Previsualiser" instead of "Preview" |
it's linked to this issue : #852 |
closes #811 closes #793 closes #809