fix(locale): Change locale code from 'fa-IR' to 'fa'

[IranMine123]

🔗 Linked issue

#4819

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

changed locale to 'fa' due to mismatch with BCP-47 standard + language code defined from i18n module is defined as 'fa'

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[benjamincanac]

As mentioned in #4819 (comment), I don't think the code is incorrect here. This could also introduce a breaking change.

You're free to define your locales as you want in @nuxtjs/i18n and also use locale fallback for such cases: https://i18n.nuxtjs.org/docs/guide/locale-fallback

@mehotkhan what do you think?

[IranMine123]

As mentioned in #4819 (comment), I don't think the code is incorrect here. This could also introduce a breaking change.

You're free to define your locales as you want in @nuxtjs/i18n and also use locale fallback for such cases: https://i18n.nuxtjs.org/docs/guide/locale-fallback

@mehotkhan what do you think?

Well for i18n module we should keep the code option to fa because it has another options for fa-IR(like iso).

Although this could be fixed by mapping fa-IR to fa but I don't think it's identical.

EDIT: also since there is no any other fa locales with different region at this time (for example fa-AF) there's no need to put the region for the code