Skip to content

Add Afrikaans as a supported language#8379

Draft
wolfen351 wants to merge 2 commits intoSonarr:v5-developfrom
wolfen351:afrikaans
Draft

Add Afrikaans as a supported language#8379
wolfen351 wants to merge 2 commits intoSonarr:v5-developfrom
wolfen351:afrikaans

Conversation

@wolfen351
Copy link
Copy Markdown

Description

Just a simple language patch

Screenshots for UI Changes

Initial Afrikaans Support

Database Migration

No

Issues Fixed or Closed by this PR

None

public static Language Urdu => new Language(50, "Urdu");
public static Language Romansh => new Language(51, "Romansh");
public static Language Georgian => new Language(52, "Georgian");
public static Language Afrikaans => new Language(53, "Afrikaans");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Test cases need to be added to src/NzbDrone.Core.Test/Languages/LanguageFixture.cs

new IsoLanguage("rm", "", "roh", Language.Romansh),
new IsoLanguage("ka", "", "kat", Language.Georgian)
new IsoLanguage("ka", "", "kat", Language.Georgian),
new IsoLanguage("af", "", "afr", Language.Afrikaans)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Test cases need to be added to src/NzbDrone.Core.Test/ParserTests/IsoLanguagesFixture.cs

@@ -0,0 +1,2238 @@
{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As I'm seeing a lot of English translations, this needs to be removed and be addressed in weblate otherwise it will state it's almost complete when in fact it's very incomplete.

Copy link
Copy Markdown
Member

@markus101 markus101 left a comment

Choose a reason for hiding this comment

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

Please take a look at B's comments and address, as-is this cannot be merged.

@markus101 markus101 marked this pull request as draft April 12, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants