Giter VIP home page Giter VIP logo

Comments (9)

tejainece avatar tejainece commented on June 15, 2024

@jaumard Could you please upgrade jaguar_query_sqflite package to 2.2.x?

One of the breaking changes is merging of Col and Field into Field. There are some minor simplifying changes too. You could use jaguar_query_postgres package for reference.

Thanks.

from jaguar_orm.

jaumard avatar jaumard commented on June 15, 2024

Sure :) let me look at this :)

from jaguar_orm.

jaumard avatar jaumard commented on June 15, 2024

@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.
jaguar_query_postgres is put as dev_dep even if not only for dev.
I say that because jaguar_query_postgres doesn't work with flutter as it use an SDK too recent for now.

from jaguar_orm.

jaumard avatar jaumard commented on June 15, 2024

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

from jaguar_orm.

tejainece avatar tejainece commented on June 15, 2024

jaguar_query_postgres is just a dev dependency. So there are some real life examples directly in the query repo. There is no actual dependence.

from jaguar_orm.

tejainece avatar tejainece commented on June 15, 2024

That change is based on how we describe it in english.

Update this table with these rows.

Insert these rows in into this table

I have fixed the Find according to your suggestions.

FInd from this table where this expression select these fields

from jaguar_orm.

tejainece avatar tejainece commented on June 15, 2024

Will also change insert statement according to your suggestions. Maybe consistency is more important that correlation with English language.

from jaguar_orm.

jaumard avatar jaumard commented on June 15, 2024

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)

from jaguar_orm.

tejainece avatar tejainece commented on June 15, 2024

Changed insert.

from jaguar_orm.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.