Skip to content

doc: export return type for syncShapeToTable#452

Merged
samwillis merged 3 commits intomainfrom
451-export-type
Dec 8, 2024
Merged

doc: export return type for syncShapeToTable#452
samwillis merged 3 commits intomainfrom
451-export-type

Conversation

@yacineb
Copy link
Contributor

@yacineb yacineb commented Dec 2, 2024

No description provided.

@yacineb yacineb requested a review from samwillis December 2, 2024 11:46
@yacineb yacineb self-assigned this Dec 2, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2024

Copy link
Collaborator

@samwillis samwillis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, thanks @yacineb

I've added a changeset. To do it in future run pnpm changeset from the root of the repo and follow the instructions.

@samwillis samwillis merged commit c3d98d5 into main Dec 8, 2024
7 checks passed
@samwillis samwillis deleted the 451-export-type branch December 8, 2024 14:53
samwillis added a commit that referenced this pull request Dec 8, 2024
* doc: export return type for syncShapeToTable

* feat: expose stream object as of for useShape hook

* Changeset

---------

Co-authored-by: Sam Willis <sam.willis@gmail.com>
samwillis added a commit that referenced this pull request Dec 9, 2024
…ters to `syncShapeToTable` (#456)

* Add `commitGranularity` parameter in the `syncShapeToTable`

* Fix tests

* Docs

* add tests for commitGranularity

* Add test of the commitThrottle option

* Fix onInitialSync and add test

* doc: export return type for syncShapeToTable (#452)

* doc: export return type for syncShapeToTable

* feat: expose stream object as of for useShape hook

* Changeset

---------

Co-authored-by: Sam Willis <sam.willis@gmail.com>

* Remove part of test thats flaky on CI

* formatting

* Fix types

* Remove transaction commit granularity until Electric has stabilised on LSN metadata

* Post review changes

---------

Co-authored-by: Yacine <ybsoft2011@live.com>
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.

2 participants