-
Notifications
You must be signed in to change notification settings - Fork 52
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
Upgrade jaguar_query_sqflite package to 2.2.x #50
Comments
@jaumard Could you please upgrade One of the breaking changes is merging of Thanks. |
Sure :) let me look at this :) |
@tejainece jaguar_query has strong dependency on jaguar_query_postgres where it should be the opposite jaguar_query shouldn't know about jaguar_query_postgres at all. |
Also you have change update(tableName) from update()..into(tableName) but Find and Insert didn't change, I don't think is good to have different notation, it should be align |
|
That change is based on how we describe it in english.
I have fixed the
|
Will also change insert statement according to your suggestions. Maybe consistency is more important that correlation with English language. |
Hum in my opinion consistency is more important, even more because Insert(tableName) don't throw any error as it's a correct statement (it will probably crash at runtime) |
Changed insert. |
No description provided.
The text was updated successfully, but these errors were encountered: