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

Unable to migrate #10

Open
rejmank opened this issue May 14, 2019 · 0 comments
Open

Unable to migrate #10

rejmank opened this issue May 14, 2019 · 0 comments

Comments

@rejmank
Copy link

rejmank commented May 14, 2019

Hello,
i try to follow your awesome lections but while setting up db i run into error.
Steps to reproduce
clone the repo
$ npm i
$ npm run db:migrate
result
`> [email protected] db:migrate C:\nodejs-open-knowledge

knex migrate:latest --knexfile ./src/database/knexfile.js

C:\nodejs-open-knowledge\node_modules\knex\lib\client.js:74
throw new Error(knex: Required configuration option 'client' is missing.);
^

Error: knex: Required configuration option 'client' is missing.
at new Client (C:\nodejs-open-knowledge\node_modules\knex\lib\client.js:74:11)
at Knex (C:\nodejs-open-knowledge\node_modules\knex\lib\knex.js:57:42)
at initKnex (C:\nodejs-open-knowledge\node_modules\knex\bin\cli.js:95:10)
at Command.commander.command.description.action (C:\nodejs-open-knowledge\node_modules\knex\bin\cli.js:190:17)
at Command.listener (C:\nodejs-open-knowledge\node_modules\commander\index.js:315:8)
at Command.emit (events.js:189:13)
at Command.parseArgs (C:\nodejs-open-knowledge\node_modules\commander\index.js:654:12)
at Command.parse (C:\nodejs-open-knowledge\node_modules\commander\index.js:474:21)
at Liftoff.invoke (C:\nodejs-open-knowledge\node_modules\knex\bin\cli.js:278:13)
at Liftoff.execute (C:\nodejs-open-knowledge\node_modules\liftoff\index.js:203:12)
at module.exports (C:\nodejs-open-knowledge\node_modules\flagged-respawn\index.js:51:3)
at Liftoff. (C:\nodejs-open-knowledge\node_modules\liftoff\index.js:195:5)
at C:\nodejs-open-knowledge\node_modules\liftoff\index.js:165:9
at C:\nodejs-open-knowledge\node_modules\v8flags\index.js:135:14
at C:\nodejs-open-knowledge\node_modules\v8flags\index.js:41:14
at C:\nodejs-open-knowledge\node_modules\v8flags\index.js:53:7`

Expected
DB should migrate

Can you please help me with this issue? I investigated it but it seems that we pass the client property into knex config

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

No branches or pull requests

1 participant