react-i18next icon indicating copy to clipboard operation
react-i18next copied to clipboard

Update `t` function types to rely on types coming from i18next

Open pedrodurek opened this issue 3 years ago • 7 comments

Dependent on https://github.com/i18next/i18next/pull/1775

We'll no longer support typescript below v4.1

Checklist

  • [x] only relevant code is changed (make a diff before you submit the PR)
  • [ ] run tests npm run test
  • [ ] tests are included

Checklist (for documentation change)

  • [ ] only relevant documentation part is changed (make a diff before you submit the PR)
  • [ ] motivation/reason is provided

pedrodurek avatar May 13 '22 23:05 pedrodurek

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar May 25 '22 22:05 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 04 '22 01:06 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 18 '22 17:06 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 10 '22 04:07 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 30 '22 20:07 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Aug 11 '22 20:08 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 09 '22 02:09 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 24 '22 09:09 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Oct 12 '22 09:10 stale[bot]

Bump, not stale! @pedrodurek what is blocking on this PR? Is there something I can do to help push it along?

For context the issue we are facing is the following, and going on the title of this PR it should fix the issue we are seeing.

image

develohpanda avatar Oct 16 '22 22:10 develohpanda

I believe this is good to go. I'll leave a request change to bump the minimum i18next once i18next/i18next#1775 is released.

Thank you @pedrodurek

v22.0.0 is now published

adrai avatar Oct 19 '22 07:10 adrai

@pedrodurek let me know when this can be merged

adrai avatar Oct 19 '22 15:10 adrai

Coverage Status

Coverage remained the same at 95.862% when pulling ad82365b098b8ef964e5cca8dd8a3d0a104eee09 on pedrodurek:make-translation-type-safe into a0eed42303bda4863a832cbd8b280d04448f2690 on i18next:master.

coveralls avatar Oct 19 '22 16:10 coveralls

Hey @adrai, this is good now. It needs to be a Major release.

pedrodurek avatar Oct 19 '22 16:10 pedrodurek

Later today (my timezone 😛), I'll update the react-i18next doc adding a reference to the i18next doc.

pedrodurek avatar Oct 19 '22 16:10 pedrodurek

@rosskevin as soon as you approve, I'll create a new major version...

btw: in case you're interested: https://github.com/i18next/next-i18next/pull/1966

adrai avatar Oct 20 '22 13:10 adrai

Thank you for your contributions... v12.0.0 has just been published.

adrai avatar Oct 20 '22 16:10 adrai

Incredible; thank you all so much for such a quick turnaround! 😍

develohpanda avatar Oct 20 '22 23:10 develohpanda