Skip to content

Conversation

Chriztiaan
Copy link
Contributor

@Chriztiaan Chriztiaan commented Sep 26, 2024

Original issue reported in dart optional sync demo powersync-ja/powersync.dart#174.
This ensures that we correctly copy over the todos with the authed user ID instead of the default local ID.
Instead of copying the local data over to the sync table verbatim (and getting the owner ID from the local table), we use the newly logged in user's ID.

Copy link

changeset-bot bot commented Sep 26, 2024

⚠️ No Changeset found

Latest commit: 76a25fa

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Chriztiaan Chriztiaan changed the title Fixed updating schema when moving todos to synced table Optional Sync Demo: Fixed updating schema when moving todos to synced table Sep 26, 2024
@Chriztiaan Chriztiaan merged commit f7db401 into main Sep 26, 2024
5 checks passed
@Chriztiaan Chriztiaan deleted the fix/optional-sync-todos branch September 26, 2024 07:20
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