Skip to content

add withSchema support#1638

Merged
ricardograca merged 1 commit into
bookshelf:masterfrom
rendongsc:master
Feb 18, 2018
Merged

add withSchema support#1638
ricardograca merged 1 commit into
bookshelf:masterfrom
rendongsc:master

Conversation

@rendongsc

@rendongsc rendongsc commented Sep 1, 2017

Copy link
Copy Markdown

add withSchema support

Our system has multiple modules, each module is a separate database schema, we hope that through an single knex instance of the operation of multiple schmea.
knexjs-withSchema

@ricardograca

Copy link
Copy Markdown
Member

Would be helpful if you filled out the issue template so we can get some context about this proposal.

@maxkoryukov

Copy link
Copy Markdown

@rendongsc I use just a few features, but in my code, I utilize withSchema and I haven't any problem. All work just fine

@ricardograca ricardograca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add the tests myself in a subsequent pull request.

@ricardograca ricardograca merged commit 54e45c1 into bookshelf:master Feb 18, 2018
weswigham pushed a commit to DefinitelyTyped/DefinitelyTyped that referenced this pull request Mar 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants