feat(i18n): allow custom interpolation options for i18n

[zleight1]

Allow for specifying custom interpolation options for i18n. This is useful for custom interpolation patterns, such as {{ and }}. Tests added, and scoped to the i18n module.

Closes #4726

🔎 Overview

Adds the ability to use custom interpolation options, such as prefix and suffix.

This allows the library to comply with the default i18next standard.

🤓 Code snippets/examples (if applicable)

configure({
  generateMessage: localize(
    'en',
    {
      messages: {
        between: 'The {{field}} field must be between 0:{{min}} and 1:{{max}}',
      },
    },
    {
      prefix: '{{',
      suffix: '}}',
    },
  ),
});

✔ Issues affected

closes #4726

[codecov]

Codecov Report

Attention: Patch coverage is 91.48936% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 89.89%. Comparing base (2a0881f) to head (ebece65).
Report is 13 commits behind head on main.

Files	Patch %	Lines
packages/i18n/src/index.ts	90.32%	3 Missing ⚠️
packages/i18n/src/utils.ts	90.90%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4727      +/-   ##
==========================================
+ Coverage   89.87%   89.89%   +0.01%     
==========================================
  Files          93       93              
  Lines        7676     7709      +33     
  Branches     1360     1362       +2     
==========================================
+ Hits         6899     6930      +31     
- Misses        770      772       +2     
  Partials        7        7              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zleight1]

If accepted I'll update the docs as well

[zleight1]

@logaretm - is there something I can do to help with making this a reality?

If there's some reason you would not want this, please let me know why so I can understand more.

Thanks!

[logaretm]

Sorry for the delay, Thank you @zleight1, This looks alright to me. I have no concerns here other than the test file seems to have changed significantly. I would have assumed this would need to add a test case or two but the changes here are too significant with some tests altered, if possible lets keep everything as-is and only add the needed test cases for this interpolation feature.

Once that is fixed we can merge this right away.

[zleight1]

If accepted I'll update the docs as well

Updated this as well, let me know if it needs anything else.

[logaretm]

Amazing, thank you for addressing my nitpicks 😛

Thanks a lot for this, merging right away and will tag a release once another PR is merged.

[zleight1]

Amazing, thank you for addressing my nitpicks 😛

Thanks a lot for this, merging right away and will tag a release once another PR is merged.

I'm all for the nitpicks, thanks for the guidance!