Skip to content

Wip knex#36

Open
BrettMatsumoto wants to merge 51 commits intodevleague:masterfrom
BrettMatsumoto:wip-knex
Open

Wip knex#36
BrettMatsumoto wants to merge 51 commits intodevleague:masterfrom
BrettMatsumoto:wip-knex

Conversation

@BrettMatsumoto
Copy link

Hey Ed, Brad and I were trying to get my articles/put route but even with method override my put form isn't hitting my put route. We looked at the form and the route and if we take out the title parameter it will just hit the post route but with the title parameter it doesn't hit any route.

Your assistance with this matter is greatly appreciated.

@taesup
Copy link
Contributor

taesup commented May 4, 2019

you checked in your node_modules. I can't help you from home with this. It craps out my browser trying to find your files.

@taesup
Copy link
Contributor

taesup commented May 4, 2019

your issue is that in your server.js file, you have your app.use(router) above your methodOverride. The routes should be the last thing in that file above you server.listen.

@BrettMatsumoto
Copy link
Author

Okay, I’ll make the change. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants