Use constructor instead of cWillMount for correct injection order

[julienben]

React Boilerplate

• You have followed our contributing guidelines
• Double-check your branch is based on dev and targets dev
• Pull request has tests (we are going for 100% coverage!)
• Code is well-commented, linted and follows project conventions
• Documentation is updated (if necessary)
• Internal code generators and templates are updated (if necessary)
• Description explains the issue/use-case resolved and auto-closes related issues

Changelog

This is an alternative to @tonyex52's fix for #2314 in his PR #2467.

Instead of making injection Promise-based (which has a few drawbacks not addressed in #2467), we simply do the injection from the constructor.

It'll guarantee that things happen in the correct order, as described by @flameddd in #2314.

All tests are passing with no changes required.

I also did the same thing for reducer injection, if only for consistency.

Also, a small doc update that was missed when we switched the default saga injection mode to DAEMON.

[coveralls]

Coverage remained the same at 100.0% when pulling f08dace on saga-injection-fix into 75dd475 on dev.

[coveralls]

Coverage remained the same at 100.0% when pulling f08dace on saga-injection-fix into 75dd475 on dev.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.