-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
[doc] Add SIRD routing documentation #6392
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
Conversation
|
Shouldn't we also add a @Inject parameter to it and describe that? That the Main Router will Inject stuff into the SIRD router? |
|
@schmitch there's nothing to inject though... |
|
yeah not here but it's possible but not documented. |
|
@schmitch something like this? |
|
yeah exactly but you dont need f() override def routes: Routes = {
case GET(p"/") => controller.index
}works |
fdc8bbd to
9783177
Compare
|
@schmitch okay done |
|
also added a note addressing #6107 |
|
yeah actually it not really addresses it. But +1 I will merge as is if tests are green |
* Add SIRD routing documentation * Add code example with call out to a controller * Add note about composing routers
* Add SIRD routing documentation * Add code example with call out to a controller * Add note about composing routers
Fixes #4586
Describes SIRD routing in context of a conf/routes file and by extending SimpleRouter.