Skip to content
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

Route in the News Tutorial Routes are ERROR all over. #1240

Closed
InsiteFX opened this issue Sep 24, 2018 · 2 comments
Closed

Route in the News Tutorial Routes are ERROR all over. #1240

InsiteFX opened this issue Sep 24, 2018 · 2 comments
Labels
bug Verified issues on the current code behavior or pull requests that will fix them

Comments

@InsiteFX
Copy link
Contributor

Doing the News Tutorial since you changed the routing of the controller.

All the routes are giving a 404 error unless I change them from get and post to add.

Also the create method is throwing a expected array error on the request.
Lonnie something is wrong in the new way the routes and requests are working.

@jim-parry jim-parry added the bug Verified issues on the current code behavior or pull requests that will fix them label Sep 24, 2018
@jim-parry jim-parry added this to the 4.0.0-alpha milestone Sep 24, 2018
@jim-parry
Copy link
Contributor

The above is part of the fix. Checking the routes now

@lonnieezell
Copy link
Member

It looks like the issue happens when a combination of get routes and add routes are used, like:

$routes->add('/', 'Home::index');
$routes->get('news', 'News::index');
$routes->get('news/(:segment)', 'News::view/$1');
$routes->add('(:any)', 'Pages::view/$1');

However, if all of the routes are changed to either get or add it works as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Verified issues on the current code behavior or pull requests that will fix them
Projects
None yet
Development

No branches or pull requests

3 participants