Skip to content

Conversation

@weswigham
Copy link
Contributor

I missed the same type in v6 when making #34335. 😢

weswigham added a commit to weswigham/react-dnd that referenced this pull request Mar 30, 2019
Fixing microsoft/TypeScript#30634 will make this constraint fail to check for most users - this type seems to have been copied from (to?) `react-redux`, which has taken in a similar change on DT:

DefinitelyTyped/DefinitelyTyped#34335
DefinitelyTyped/DefinitelyTyped#34339
@typescript-bot typescript-bot added Popular package This PR affects a popular package (as counted by NPM download counts). Awaiting reviewer feedback labels Mar 30, 2019
@typescript-bot
Copy link
Contributor

@weswigham Thank you for submitting this PR!

🔔 @tkqubo @kenzierocks @clayne11 @tansongyang @NicholasBoll @mDibyo @Kallikrein @val1984 @jrakotoharisoa @apapirovski @surgeboris @soerenbf @pdeva - please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

If no reviewer appears after a week, a DefinitelyTyped maintainer will review the PR instead.

@weswigham weswigham merged commit 0617f9d into master Mar 30, 2019
@typescript-bot
Copy link
Contributor

I just published @types/react-redux@6.0.14 to npm.

darthtrevino pushed a commit to react-dnd/react-dnd that referenced this pull request Mar 31, 2019
Fixing microsoft/TypeScript#30634 will make this constraint fail to check for most users - this type seems to have been copied from (to?) `react-redux`, which has taken in a similar change on DT:

DefinitelyTyped/DefinitelyTyped#34335
DefinitelyTyped/DefinitelyTyped#34339
alesn pushed a commit to alesn/DefinitelyTyped that referenced this pull request Apr 23, 2019
@elibarzilay elibarzilay deleted the fix-rr-v6 branch November 28, 2020 23:58
darthtrevino pushed a commit to react-dnd/react-dnd that referenced this pull request Feb 3, 2022
Fixing microsoft/TypeScript#30634 will make this constraint fail to check for most users - this type seems to have been copied from (to?) `react-redux`, which has taken in a similar change on DT:

DefinitelyTyped/DefinitelyTyped#34335
DefinitelyTyped/DefinitelyTyped#34339
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Popular package This PR affects a popular package (as counted by NPM download counts).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants