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

v1.0.0 Roadmap #122

Closed
3 tasks done
timche opened this issue Aug 31, 2016 · 13 comments
Closed
3 tasks done

v1.0.0 Roadmap #122

timche opened this issue Aug 31, 2016 · 13 comments
Assignees

Comments

@timche
Copy link
Member

timche commented Aug 31, 2016

With v1.0.0 we are releasing our first version that comes with breaking changes:

Breaking Changes:

@timche
Copy link
Member Author

timche commented Aug 31, 2016

I would also like to introduce Flowtype with this version as it is getting popular and already being used in many React application as well as in create-react-app.

cc @yangmillstheory

@yangmillstheory
Copy link
Contributor

Sounds good.

I remember you had a great idea to clean up the documentation too. Just wanted to make sure that's not forgotten (not sure which epic it goes in).

@timche
Copy link
Member Author

timche commented Aug 31, 2016

Oh, yeah, documentation cleanup/improvement would also be nice. I will add it.

@timche
Copy link
Member Author

timche commented Sep 1, 2016

Because #115 seems to be more relevant than expected, we can release that as only change in v1.0.0 and everything else in v1.1.0 to speed up the release.

cc @yangmillstheory

@yangmillstheory
Copy link
Contributor

That sounds good. Are you going to take #115?

@timche
Copy link
Member Author

timche commented Sep 1, 2016

Yes @yangmillstheory

@yangmillstheory
Copy link
Contributor

@timche can I take care of #47 in v1.0.0?

@timche
Copy link
Member Author

timche commented Sep 3, 2016

That would be huge breaking change. Could be sth for v2.

@yangmillstheory
Copy link
Contributor

What about #18?

I'm not sure if this would also be breaking actually, since I think Redux throws errors on missing default state.

#18 just ensures the errors are thrown earlier.

@timche
Copy link
Member Author

timche commented Sep 6, 2016

Yes, this is actually breaking, since we are making this required, even it was throwing an error before.

@yangmillstheory
Copy link
Contributor

yangmillstheory commented Sep 6, 2016

OK. Please see #127.

@yangmillstheory
Copy link
Contributor

Can we add type-checking of payloadCreator and metaCreator to this?

See the first example in #128 (comment).

@yangmillstheory
Copy link
Contributor

I've ported this roadmap issue over to https://github.com/acdlite/redux-actions/projects/1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants