Skip to content

[TS]: Primary keys for query builder views#4573

Open
joshua-spacetime wants to merge 10 commits intojoshua/rust/transfer-pk-to-viewfrom
joshua/ts/transfer-pk-to-view
Open

[TS]: Primary keys for query builder views#4573
joshua-spacetime wants to merge 10 commits intojoshua/rust/transfer-pk-to-viewfrom
joshua/ts/transfer-pk-to-view

Conversation

@joshua-spacetime
Copy link
Collaborator

@joshua-spacetime joshua-spacetime commented Mar 6, 2026

Description of Changes

Equivalent changes to #4572, but for typescript, and minus any server side changes.

Query builder views can now be annotated with the .query() return type, in addition to the pre-existing .array() return type. If .query() is used, the view will have a primary key if the underlying table that it returns has one. This means client-side onUpdate callbacks will be generated.

API and ABI breaking changes

None. t.array() return type views are still supported.

Expected complexity level and risk

3

Testing

The equivalent typescript versions of the sdk tests added in #4572.

@joshua-spacetime joshua-spacetime force-pushed the joshua/ts/transfer-pk-to-view branch from 85c6475 to 1494900 Compare March 6, 2026 05:54
@joshua-spacetime joshua-spacetime force-pushed the joshua/rust/transfer-pk-to-view branch from 2088fbc to ff7bec7 Compare March 10, 2026 16:28
@joshua-spacetime joshua-spacetime force-pushed the joshua/ts/transfer-pk-to-view branch from 1494900 to c777e7b Compare March 10, 2026 16:33
@joshua-spacetime joshua-spacetime removed the request for review from cloutiertyler March 10, 2026 16:43
@joshua-spacetime joshua-spacetime force-pushed the joshua/rust/transfer-pk-to-view branch from ff7bec7 to 1be82af Compare March 10, 2026 16:50
@joshua-spacetime joshua-spacetime force-pushed the joshua/ts/transfer-pk-to-view branch from c777e7b to e9e6dd2 Compare March 10, 2026 16:50
@joshua-spacetime joshua-spacetime force-pushed the joshua/rust/transfer-pk-to-view branch from 1be82af to c8d8630 Compare March 11, 2026 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant