anidl / multi-downloader-nx

Downloader for Crunchyroll, Hidive, AnimeOnegai, and AnimationDigitalNetwork with CLI and GUI
MIT License
267 stars 44 forks source link

[Feedback]: Inconsistencies between country codes format in configuration makes it easy to crash the GUI app #674

Open abaurens opened 5 months ago

abaurens commented 5 months ago

Type

Both

Suggestion

In the cli-defaults.yml config, the dlsubs field expects a 2-char ISO 3166-1 country code while dubLang and defaultAudio both expects a 3-char ISO 3166-2 one.

Even tho this is documented in their respective sections of the documentation:

It is missleading when working with the GUI version and editing the config manually, as the default for dlsubs is all which gives no indication about the expected format (unlike dubLang and defaultAudio which both defaults to jpn).

This results in easy mistakes which makes the app silently crash upon service selection because it doesn't recognize the 3-char code.


Here are a few suggestions that could fix the issue, even tho I don't know if they are all technically applicable: