Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

feat(de-translations): added de-DE translations to the UI #441

Merged
merged 13 commits into from
Mar 12, 2020

Conversation

priscilawebdev
Copy link
Contributor

Type: feat

Description:

follow up of #432

image

@codecov
Copy link

codecov bot commented Mar 9, 2020

Codecov Report

Merging #441 into master will decrease coverage by 0.53%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #441      +/-   ##
==========================================
- Coverage   88.08%   87.55%   -0.54%     
==========================================
  Files         140      140              
  Lines         982      988       +6     
  Branches      204      204              
==========================================
  Hits          865      865              
- Misses        106      112       +6     
  Partials       11       11              
Impacted Files Coverage Δ
i18n/config.ts 100.00% <ø> (ø)
src/App/load-dayjs-locale.ts 33.33% <0.00%> (-13.34%) ⬇️

@juanpicado
Copy link
Member

LGTM, but I'd rather trust on @DanielRuf 👍

Copy link

@jan-auer jan-auer left a comment

Choose a reason for hiding this comment

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

Sounds really good, here are a few suggestions.

i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
i18n/translations/de-AT.json Outdated Show resolved Hide resolved
@priscilawebdev
Copy link
Contributor Author

@jan-auer Thank you very much for all the suggestions 😉 I've applied all of them.

@priscilawebdev priscilawebdev changed the title feat(de-translations): added de-AT translations to the UI feat(de-translations): added de-DE translations to the UI Mar 12, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@verdacciobot
Copy link

Thanks for your PR, the @verdaccio/ui package will be accessible from here for testing purposes:

npm install @verdaccio/[email protected] --registry https://registry.verdaccio.org

@@ -32,6 +32,18 @@ function loadDayJSLocale() {
dayjs.locale('pt-br');
}
break;
case 'de':
Copy link
Member

Choose a reason for hiding this comment

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

would not be de-de?

Copy link
Contributor Author

@priscilawebdev priscilawebdev Mar 12, 2020

Choose a reason for hiding this comment

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

nope... de is the default one 😉 Please see: https://github.com/iamkun/dayjs/tree/dev/src/locale

@priscilawebdev priscilawebdev merged commit e27d59b into master Mar 12, 2020
@delete-merged-branch delete-merged-branch bot deleted the feat/added-de-at-translations branch March 12, 2020 10:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants