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

escaping boolean to 0 or 1 for mssql #4621

Closed
wants to merge 2 commits into
base: master
from

Conversation

2 participants
@jonnolen
Contributor

jonnolen commented Oct 7, 2015

resolves #4619,

looking for feedback on where the tests live that might be covering this old behavior.

Update sql-string.js
escaping boolean to be 0 or 1 for mssql dialect.
@jonnolen

This comment has been minimized.

Contributor

jonnolen commented Oct 7, 2015

i'll get that straightened out tonight.

@janmeier

This comment has been minimized.

Member

janmeier commented Oct 12, 2015

Thanks! :)

Squashed and changelog added in 0de4046

@janmeier janmeier closed this Oct 12, 2015

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