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

Your changes has been incorporated into v3 branch #1

Closed
wants to merge 469 commits into from

Conversation

@techno-express
Copy link

@techno-express techno-express commented Mar 23, 2018

Hi,
Just to let you know, i have forked and added onto most of your work on ezSQL. It has now been added into an branch and eventually will be part of the main repo.

I am one of the collaborators to the project now. I was in the process of write my own unit tests and other major rewrites, then i came upon your pull request, seen it closed with no reasons. A lot of great changes too.

techno-express added 14 commits Apr 5, 2018
… in vendor class driver for all values passed
… statements change on entering Query method call
…dors prepare functions for arguments variables supplied, testing on ms sql server
…tParamaters functions, added query_prepared for mysql, sqlite3 and pdo

These functions will allow supplied variables, values, string or array, to be auto converted to placeholders then bind and executed, returning an result set.
…etup basic escape function template use if none available by sql vendor
@techno-express techno-express deleted the techno-express:master branch Sep 5, 2018
@sjstoelting
Copy link
Owner

@sjstoelting sjstoelting commented Oct 11, 2018

Hi @techno-express
I merged the ezSQL master branch from upstream and switched the default branch to master.
In my branch ezSQL-3.0 I've added the information to use the original sources of ezSQL. In addition I published that as news on my website, I've seen several page accesses to my ezSQL news list.

Thanks for you work merging my stuff into ezSQL back again.

Stefanie

@techno-express
Copy link
Author

@techno-express techno-express commented Oct 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.