-
Notifications
You must be signed in to change notification settings - Fork 363
Conversation
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.
I don't know Danish but I ran it through Google Translate and it seems to check out 😎.
One small fix and we'll be good to go! Thanks for the PR.
docs/theme-scaffold/index.md
Outdated
@@ -154,7 +154,7 @@ Slate projects include a `vendor` folder where theme developers are encouraged t | |||
|
|||
## Translation keys | |||
|
|||
Slate ships with six languages — English, French (Canadian), Spanish, German, and two dialects of Portuguese (Brazilian and European). Translation keys for these languages can be found in the `locales` folder. In your theme files, translated strings can be retrieved with the Liquid translation filter `t`. | |||
Slate ships with six languages — English, French (Canadian), Spanish, German, Danish, and two dialects of Portuguese (Brazilian and European). Translation keys for these languages can be found in the `locales` folder. In your theme files, translated strings can be retrieved with the Liquid translation filter `t`. |
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.
"Slate ships with seven languages"... soon 😄
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.
Dang.. Missed that one 😸 Now updated along with a typo fix.
Thanks @madsobel 👍 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
What are you trying to accomplish with this PR?
I'm currently creating a store in Danish, and though I'd add my local translation.
Checklist
For contributors:
For maintainers:
package.json
version in a separate PR, if applicable.