You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Unfortunately there is not much documentation about such integration and I'm not 100% sure by design you intend to have it working in such direction, so it would be nice to have some discussion about potential issues that this PR would create.
The text was updated successfully, but these errors were encountered:
Hi! 馃憢
Firstly, thanks for your work on this project! 馃檪
Today I used patch-package to patch
knex@3.0.1
for the project I'm working on.Here is the diff that solved my problem:
This issue body was partially generated by patch-package.
Abstract about the problem
When using the
sqlite3
adapter, the date / datetime values are not written properly to the database.The patch from #5272 only partially addresses the issue since it only works for inserts but not for updates.
This issue contains a proposal for fixing the issue by adding these changes:
Date
to a date string prior writing to the database.Unfortunately there is not much documentation about such integration and I'm not 100% sure by design you intend to have it working in such direction, so it would be nice to have some discussion about potential issues that this PR would create.
The text was updated successfully, but these errors were encountered: