Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Only send parameters with query if they are passed #89

Merged
merged 1 commit into from
May 11, 2022
Merged

Only send parameters with query if they are passed #89

merged 1 commit into from
May 11, 2022

Conversation

leonaves
Copy link
Contributor

Closes #88

@leonaves
Copy link
Contributor Author

@brianc are you happy with this?

@leonaves
Copy link
Contributor Author

Any updates on this? @brianc

@brianc
Copy link
Owner

brianc commented May 18, 2020

hey sorry I just saw this - I get an avalanche of email and it must have slipped through in the past. I'll try & get this merged today.

@leonaves
Copy link
Contributor Author

Great thanks, and no worries, same happens to me, was just bumping in case. :)

@leonaves
Copy link
Contributor Author

@brianc

2 similar comments
@leonaves
Copy link
Contributor Author

@brianc

@leonaves
Copy link
Contributor Author

leonaves commented Jul 1, 2021

@brianc

@Macavirus
Copy link

@brianc is this one possible to merge? Not sure what we're waiting on here or just forgot :)

@leonaves
Copy link
Contributor Author

@brianc Hey not sure if you've seen this

@leonaves leonaves mentioned this pull request May 11, 2022
Merged
@brianc
Copy link
Owner

brianc commented May 11, 2022

my bad - i don't have notifications on on this repo...I'll merge this now & try to get a release out next time I push out the pg package.

@brianc brianc merged commit 8e333ab into brianc:master May 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@charmander charmander charmander approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot insert multiple commands into a prepared statement
4 participants
@leonaves @brianc @Macavirus @charmander