-
-
Notifications
You must be signed in to change notification settings - Fork 767
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
RQB v2 #3974
base: beta
Are you sure you want to change the base?
RQB v2 #3974
Conversation
…to rqb-v2-orm36
…` field, RQB v1 entities to `_relations.ts`, added template to SQLWrapper for return type inferrence, switched RQB-related local tests & type tests to use `._query`
…thReplicas()` compatibility (to be reworked), switched tests to use `_query` method for RQB v1 tests
… fixed, simplified query builders, added JSON parsing for top level SQLite response mapper, fixed aggregated relation count sql
… relational configs, resolved potential type overlaps with similar table names from different schemas on query results, resolved relation builders allowing usage of columns from table with same name from different schema
…nt relations' `where.$raw` callback
…ypes to tables instead of column records, relational tests v2 for postgresql, updated relations .where field names, added error on nothing selected from table in relational queries v2
…to rqb-v2-orm36
…http `$withAuth` rqbv2 tests
…ntity kinds, new `orderBy` api
…RQB V2, removed unnecessary argument from `WithContainer` generics
…ional relation results, added forced relation optionality for queries with additional filters, fixed libsql rqb pre-mapping and transaction result mapping, added libsql RQBv2 tests, added new RQB test cases related to aforementioned changes
…joins and `json_arrayagg()`'s `order by` order preservation, removed unused code, added error throw to incomplete RQBv2 feature `.through()`
…regated relations
…d planetscale tests for RQB v2, added common RQB tests for pg dialect drivers
…non-optional relations, defaulted relations to optional, allowed full booleans in RQB query config's `with` section, moved `coalesce` in pg RQB for consistency
…proxy session RQB fixes, fixed missing `TRelations` arguments in generics
…plified RQB column builder for `MySQL`
…, RQBv2 schema tests
…lback getting columns passed to first argument instead of full table
…to rqb-v2-orm36
Great work! |
this problem does not exist is rqb v2 |
This is so exciting. What a coincidence I'm here only 3 minutes after the latest check!! |
…itched `sql-js` prepare logic to query-builder-side preparation instead of db-side because of the need to free connections manually
Oh my lawd this is nice <3 |
any ETA on this? |
…ble` class' symbols, supported `View` in `getTableUniqueName(table)`, fixed `like`-type operators not accepting `SQLWrapper`, fixed aliased `View`-s getting prefixed with schema name in `sql` builder, fixed RQBv2 issue occuring when root-level table's name matches with queried relation name by aliasing root-level table as well
…added AWS Data Api tests for RQBv2
…to rqb-v2-orm36
…e-error to incompatible with rqb views on relations builder \& relations helper, removed `.aggs` from relational query builder for rework
@Sukairo-02 u da real mvp |
Features:
sql
prefixing aliased tables with schema namesgetTableUniqueName(table)
SQLWrapper
neon-http-batch
testssql-js
prepare logic to query-builder-side preparation instead of db-side because of the need to.free()
connections manuallyView
ingetTableUniqueName(table)
Table
symbols toView
classBreaking changes:
singlestore
's, gained 2 additional generic arguments for RQB v2 queriesDrizzleConfig
generic withTRelations
argument andrelations: TRelations
fielddb.query
=>db._query
drizzle-orm
,drizzle-orm/relations
todrizzle-orm/_relations
with RQB v2's alternatives (if such exist) being put in their place:${dialect}-core/query-builders/query
files were moved to${dialect}-core/query-builders/_query
with RQB v2's alternatives being put in their place.free()
fromsql-js