-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
update(doc): solution for Feature combineReducers and AOT #1113
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some typos
docs/store/api.md
Outdated
configuration option to provide an array of meta-reducers that are composed from right to left. | ||
@ngrx/store composes your map of reducers into a single reducer. | ||
|
||
> Developers can think of metaReducers as hooks into the action->reducer pipeline. MetaReducers allow develoeprs to pre-process actions actions before *normal* reducers are invoked. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
allow developers to pre-process actions actions before
docs/store/api.md
Outdated
and then the following will work with AOT: | ||
|
||
```ts | ||
import { reducerA } from './+state/a.reducer'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import { reducerA } from './state/a.reducer';
(without the + sign)
@tdeschryver if you want to just make those couple of edits here we can land this. |
@brandonroberts Done! |
The angular compiler needs statically analyzes code during AOT. As such you can’t call functions like
combineReducers()
when defining metadata in the NgModule.Refs #306