Igor Savin

Results 407 comments of Igor Savin

Can you please add full integration test?

@gggdomi You know that you can disable transaction, right? See config options in http://knexjs.org/#Migrations-API

Can you check if there are no conflicting versions of knex being used? If you take a look at client. js:299, you'll see it is deleting afterCreate before passing further.

@richardgirges Can you submit PR with failing test for this?

@ConnorDuvalS Could you please add integration test making use of this field in some way?

@NguyenTheSon Could you provide more info?

@asteinarson Do you have an idea what needs to be tweaked from Knex export side for that to work? Would love to support it, but wonder if it's even possible...

@asteinarson Well, I know exactly which PR affected this behaviour: https://github.com/knex/knex/pull/4181 However, I don't think it became worse. Previously named imports were not supported at all, the only option was...

it would be a semver major if default is "no"