-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
en-GB Locale. Closes #1305 #1307
Conversation
✅ Deploy Preview for dashy-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Changes preview: |
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, thanks @14sprouj :)
There's just one small thing left to do, which is importing your new locale into /src/utils/languages.js
- here's a guide on how to do this.
Something like this:
import enGB from '@/assets/locales/en-GB.json';
....
{ // English, the OG edition
name: 'British English',
code: 'en-GB',
locale: enGB,
flag: '🇬🇧',
},
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.
Hiya @14sprouj
I can't make this change on your branch from my end, as I don't think you ticked the Allow Maintainer Edits option in your PR. But before this can be merged, you just need to add the following into /src/utils/languages.js
import enGB from '@/assets/locales/en-GB.json';
and
{ // British English
name: 'British English',
code: 'en-GB',
locale: enGB,
flag: '🏴',
},
There's a guide on how to do this here, but let me know if you still need any help
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.
Where there's no change between the value in the default locale (en
/ American English), we don't need to include anything, as it'll just be inherited.
And it's actually best to just leave out these duplicate values, so that if anything needs updating it won't need to be modified in two places.
So this file can actually just look something like this:
{
"theme-maker": {
"reset-toast": "Custom Colours for {theme} Removed"
},
"widgets": {
"gluetun-status": {
"organization": "Organisation"
},
"nextcloud": {
"memory-utilisation": "memory utilisation"
}
}
}
Hi @14sprouj - I'm going to close this for now, as not heard back. But let me know below if you want to come back to it, and I'll re-open. |
Thank you for contributing to Dashy! So that your PR can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category:
Overview
Issue Number (if applicable) #1305
New Vars (if applicable)
Screenshot (if applicable)
Code Quality Checklist (Please complete)