Warning: a promise was created in a handler but not returned from it #1388
Labels
Comments
Reverting back to knex@0.10.0 removes the warnings, so it's probably related to the Bluebird upgrade done for knex@0.11.0. |
@svileng Hm. Sounds like the fix @bendrucker applied is not appropriate for the newer version of Bluebird. What do you think @wubzz? |
I'm experiencing the same bug, and fixed it temporarily by switching to v0.10.0 also. |
ezmiller
added a commit
to ezmiller/intro-to-bookshelf
that referenced
this issue
May 19, 2016
The newer version has a bug related to Promises and migrations that's still being worked out. See knex/knex#1388
ezmiller
added a commit
to ezmiller/intro-to-bookshelf
that referenced
this issue
May 19, 2016
The newer version has a bug related to Promises and migrations that's still being worked out. See knex/knex#1388
agdevbridge
added a commit
to agdevbridge/knex
that referenced
this issue
May 30, 2016
This would fix Warning: a promise was created in a handler but not returned from it Issue: knex#1388
pletcher
added a commit
to learn-co-curriculum/node-js-intro-to-bookshelf
that referenced
this issue
Jun 21, 2016
* Add test suite * Add blog_server stub * Remove pg dependency * Add version test for knex The newer version has a bug related to Promises and migrations that's still being worked out. See knex/knex#1388 * Modify server stub for migrations * Change commenting * Reorganize tests * Add comment starting queue * Remove unneeded code * Add beginnings of README * Add changes to README * Add new seciton header * Move text around * Fix name * Make minor language fixes * Add last bit of README * Fix spec * Add some revisions * Add blogging graphic * Fix markdown for blog graphic * Add minor revisions * Add resources * Fix minor typos Add convenience links in first paragraph to Knex and Bookshelf (I hope that's okay -- feel free to change if not) * Omit done for synchronous tests * Isolate server tests
pletcher
added a commit
to learn-co-curriculum/node-js-intro-to-bookshelf
that referenced
this issue
Jun 21, 2016
* Add package.json with dependencies * Add package.json test * Add beginnings of blog server * Add beginnings of blog server tests * Change name to models test * Add server * Add script to run server * Start server using root hooks * Add basic create user endpt * Add server test suite with create user test * Add a comment * Rename test files to set run order * Fix test name * Change method of setting env * Change tearDown to return promise * Strengthen test * Fix destroySchema * Remove uneeded env setting * Add posts model and tests * Change user endpt to return id only * Add create user endpt and test * Save user id instead of fetching * Add get user endpt and test * Add GET /post/id endpt plus test * Add comments model and tests * Add create comment endpt plus tests This commit also reconfigures the relationships a bit to use default relationship field names. * Remove send data on GET tests * Change test name * Add some error handling tests * Add 500 error handling * Add endpt to get all posts plus test * Clean up * Move code sections around * Add pg to package.json test * Add migrations implementation * Remove pg dependency * Add version test for knex The newer version has a bug related to Promises and migrations that's still being worked out. See knex/knex#1388 * Reorganize tests * Modify method of exporting * Change commenting * Isolate server tests * Add curly braces on single-line ifs
elhigu
added a commit
that referenced
this issue
Oct 19, 2016
I believe this can be closed out when the next release occurs (currently 0.12.5) |
tgriesser
added a commit
that referenced
this issue
Oct 19, 2016
tgriesser
added a commit
that referenced
this issue
Oct 19, 2016
RubenSlabbert
added a commit
to RubenSlabbert/knex
that referenced
this issue
Nov 9, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm trying out knex and migrations for the first time, and I'm getting a few of those after I run
migrate:latest
:Here's my package.json:
And the migration I've created:
I also tried returning a Promise from
knex.schema
directly after I callcreateTable
but that didn't make a difference. Any ideas what might be the problem? Thanks!Note: could be related to petkaantonov/bluebird/issues/482
The text was updated successfully, but these errors were encountered: