-
-
Notifications
You must be signed in to change notification settings - Fork 287
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
[Feature Request]Firebird Support #73
Comments
Hey @LiPengfei19820619, could you please submit a pull request of this feature? As you are a user that understands the needs for Firebird, you could follow this commit 6cc9774 and use this lib hgourvest/node-firebird to accomplish the same result that I've got with Postgres. Is it ok? As you can see at the commit, it's not difficult, it's almost to define the queries it needs to run. Thanks! |
I did the Firebird Interface implementation. As I have no experience with extensions and do not know how to test. Tanks |
@LiPengfei19820619 can you test the open PR? I believe it's already working, I've updated the references to the new code. Thanks! |
Closing. Needs users to test it. If someone want to move forward with it, here is the PR #123 and the branch with the code is |
Issue Type
Would you please support Firebird Database? Thanks.
The text was updated successfully, but these errors were encountered: