Skip to content
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

Ensure there's the proper number of parameter replacements. #19

Merged
merged 1 commit into from
Oct 26, 2020

Conversation

clokep
Copy link
Member

@clokep clokep commented Oct 26, 2020

Fixes #18 by fixing the insert query added in #17 to have the proper number of parameter replacements to match the inserted data.

@clokep clokep requested a review from neilisfragile October 26, 2020 13:18
Copy link
Contributor

@neilisfragile neilisfragile left a comment

Choose a reason for hiding this comment

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

lgtm

@clokep clokep merged commit d970da5 into master Oct 26, 2020
@clokep clokep deleted the clokep/fix-aggregate-bug branch October 26, 2020 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aggregate script failing when converting arguments
2 participants