-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Improving setup routes comments #22
Improving setup routes comments #22
Conversation
* $app->post('/album', [App\Handler\AlbumCreateHandler::class], 'album.create'); | ||
* $app->put('/album/:id', [App\Handler\AlbumUpdateHandler::class], 'album.put'); | ||
* $app->patch('/album/:id', [App\Handler\AlbumUpdateHandler::class], 'album.patch'); | ||
* $app->delete('/album/:id', [App\Handler\AlbumDeleteHandler::class], 'album.delete'); |
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.
@peter279k Sorry for requesting a change after approval, with that we talk on antidot-framework/antidot-framework#30, I prefer to remove the next lines until it is implemented as you suggested.
*
* Or with multiple request methods:
*
* $app->route('/contact', [App\Handler\ContactHandler::class], ['GET', 'POST', ...], 'contact');
15ed7de
to
76b7243
Compare
router/routes.php
Outdated
* | ||
* Or with multiple request methods: | ||
* | ||
* $app->route('/contact', App\Handler\ContactHandler::class, ['GET', 'POST', ...], 'contact'); | ||
* $app->route('/contact', [App\Handler\ContactHandler::class], ['GET', 'POST', ...], 'contact'); |
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.
@peter279k My apologies, if you want I can make myself, I want to mean to delete this last comment block as in https://github.com/antidot-framework/reactive-antidot-starter/pull/12/files
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.
Thanks. Deleted.
76b7243
to
7f3cae1
Compare
Changed log