Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Types enchancements #295

Merged
merged 14 commits into from
Apr 17, 2021
Merged

Types enchancements #295

merged 14 commits into from
Apr 17, 2021

Conversation

kirill-konshin
Copy link
Owner

@kirill-konshin kirill-konshin commented Oct 29, 2020

Fix #280
Fix #207
Fix #240
Fix #226
Fix #293
Based on #247

@richterk
Copy link

richterk commented Nov 12, 2020

Very interested in this change, as I think it may potentially fix the featureRequest I just posted. @kirill-konshin. Overall, the solution looks pretty good, though it does expose an API and a frame of execution that can be... quite verbose.

README.md Outdated Show resolved Hide resolved
kirill-konshin and others added 3 commits December 10, 2020 22:58
Co-authored-by: Melanie Seltzer <melleh11@gmail.com>
@kirill-konshin kirill-konshin force-pushed the types-enchancements branch 13 times, most recently from f364875 to e006a0d Compare December 12, 2020 03:20
@kirill-konshin kirill-konshin force-pushed the types-enchancements branch 4 times, most recently from b77d4da to 0c2ebd6 Compare December 12, 2020 03:35
@kirill-konshin
Copy link
Owner Author

@melanieseltzer @richterk @Jekins @khaibula @GVanderLugt https://github.com/kirill-konshin/next-redux-wrapper/releases/tag/7.0.0-rc.1 please help me to test this in the wild. Make sure you follow the updated API.

Copy link
Contributor

@melanieseltzer melanieseltzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please help me to test this in the wild. Make sure you follow the updated API.

Worked great for me 👍 Just had some comments about typing from what I discovered when integrating it... might be useful for the demo.

packages/demo-dynamic/store.tsx Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated
export type AppThunk<ReturnType = void> = ThunkAction<ReturnType, AppState, unknown, Action>;
```

As you see in this example ``
Copy link

@nikichv nikichv Feb 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is the example? 😅

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like something was lost here )

@kirill-konshin kirill-konshin merged commit e2255c2 into master Apr 17, 2021
@kirill-konshin kirill-konshin deleted the types-enchancements branch June 2, 2021 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment