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

Add queriesRunner #48

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add queriesRunner #48

wants to merge 1 commit into from

Conversation

crianonim
Copy link

Function that let's you run an array of queries synchronously with optional parameters

Dear DFIO,
I am suggesting a solution to one of your problems. I know you already solved it differently, but maybe just have a look. This way you still can execute your queries in order and have them all used parametrized queries. I read that Postgres executes one at the time anyway.
This is your project so I would expect you to instead just merging this PR, just check out my branch and play with it copy,paste etc. without my commit. That is if you like my solution and more importantly if you want to continue this project.
This problem would be solved even easier (and be more readable) if you didn't mind playing with Promises... (I can elaborate if you care)

Function that let's you run an array of queries synchronously with optional parameters
@jackbridger
Copy link
Contributor

love the way you wrapped the variables and the query in an array

Will try and spend some more time looking at this to make sure I properly understand everything

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.

2 participants