Skip to content
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

fix(norwegian_locale): change the i18n locale.name for nb-NO #567

Merged
merged 1 commit into from
Jan 15, 2021

Conversation

AlfHou
Copy link
Contributor

@AlfHou AlfHou commented Jan 15, 2021

Change the locale name of nb-NO from 'Bokmål' to 'Norsk'

Fixes #559

Change the locale name of nb-NO from 'Bokmål' to 'Norsk'
@codecov-io
Copy link

Codecov Report

Merging #567 (241e529) into master (ef41d6d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #567   +/-   ##
======================================
  Coverage    5.40%   5.40%           
======================================
  Files         102     102           
  Lines        2795    2795           
  Branches      425     425           
======================================
  Hits          151     151           
  Misses       2637    2637           
  Partials        7       7           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ef41d6d...241e529. Read the comment docs.

@sonarcloud
Copy link

sonarcloud bot commented Jan 15, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ferferga
Copy link
Member

Shouldn't we add nynorsk as indicated in the linked issue?

We might not be able to translate it, but somebody can take the translations over in Weblate.

@@ -155,7 +155,7 @@ const config: NuxtConfig = {
{ code: 'en', iso: 'en-US', name: 'English', file: 'en-US.json' },
{ code: 'es', iso: 'es-ES', name: 'Español (España)', file: 'es.json' },
{ code: 'fr', iso: 'fr-FR', name: 'Français', file: 'fr-FR.json' },
{ code: 'nb', iso: 'nb-NO', name: 'Bokmål', file: 'nb_NO.json' },
{ code: 'nb', iso: 'nb-NO', name: 'Norsk', file: 'nb_NO.json' },
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{ code: 'nb', iso: 'nb-NO', name: 'Norsk', file: 'nb_NO.json' },
{ code: 'nb', iso: 'nb-NO', name: 'Norsk bokmål', file: 'nb_NO.json' },

This is the proper name for it. As there are two variants of "Norsk", we should explicitly mention which one it is.

@heyhippari
Copy link
Contributor

heyhippari commented Jan 15, 2021

Shouldn't we add nynorsk as indicated in the linked issue?

We'd need to wait until someone on Weblate translates it, then we add it. It's pretty useless to add an empty language just for the sake of it.

Copy link
Contributor

@heyhippari heyhippari left a comment

Choose a reason for hiding this comment

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

Saw the discussion about this on the other issue, so it's all good. We can change it if someone comes around and starts doing a Nynorsk translation.

@heyhippari heyhippari merged commit fb182fb into jellyfin:master Jan 15, 2021
@hawken93
Copy link
Contributor

Nice 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Language selector shows only "Bokmål", not "Norsk" or "Norsk Bokmål"
5 participants