lb(fix): remove multiple component binding sugar #619
Conversation
shimks
commented
Feb 13, 2018
shimks
commented
- connected to Decouple wiring of implementation bits from application configuration loopback-next#742
pages/en/lb4/Application.md
Outdated
| ``` | ||
| The components collection allows bulk binding of component constructors within | ||
| The components collection allows binding of component constructors within |
There was a problem hiding this comment.
It's no longer a collection ... but idk what else you can change it to.
pages/en/lb4/Application.md
Outdated
| ``` | ||
| Much like the components collection, the controllers collection allows bulk | ||
| binding of [Controllers](Controllers.html) to | ||
| Much like the components collection, the controllers collection allows binding of [Controllers](Controllers.html) to |
| #### Controllers | ||
| ```ts | ||
| app.controller(FooController); | ||
| app.controller(BarController); |
There was a problem hiding this comment.
does it mean app.controller no longer takes in an array of controller ctor?
There was a problem hiding this comment.
It never did in the first place. The change is being made to a WIP PR that got accidentally merged, and since that feature (registration of array of controllers) is no longer being implemented, I'm reverting that portion of the PR.
| #### Servers | ||
| ```ts | ||
| app.server(RestServer); | ||
| app.servers([MyServer, GrpcServer]); |
There was a problem hiding this comment.
Should we eventually get rid of app.servers() (the plural version) for consistency? It's out of scope of this pull request though.
There was a problem hiding this comment.
I think that would be a good idea ... at least till we have plans to re-visit multi-server support.
