Skip to content

Merge master into alpha release #122

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

Merged
merged 56 commits into from
Jul 17, 2024

Conversation

mugikhan
Copy link
Contributor

No description provided.

cahofmeyr and others added 30 commits May 30, 2024 08:56
Readme update: Messaging/description of powersync
* Initial extension testing.

* Replace schema logic with rust extension.

* Use extension for saving operations.

* Auto tx_id.

* Use sync_local op.

* Validate checksums using extension.

* clear_remove_ops

* delete_pending_buckets

* delete_bucket

* Fix deadlock on closing of connection.

* WIP.

* Update/fix tests.

* Add view name override.

* Retry opening database if locked.

* Fix lint issues.

* Use higher-level `updateSchema` in tests; test viewName alias.

* Load extension for android and iOS

* Load extension for iOS, MacOS and Android

* Include powersync_flutter_libs

* Rename android package

* Update windows and linux extension scripts

* Download binary in cmake

* Fix arch in cmake

* Fix binary download

* Load extension on windows

* Clean up readme and changelog

* Remove test code

* Update Abi check for current platform

* Clean up unused files

* Update path dependencies

* Test downloading powersync binary for tests

* Fix download for linux binary

* Add binary download for demos workflow

* Download latest binaries

* Bump version of powersync

* Bump minor version

---------

Co-authored-by: Ralf Kistner <[email protected]>
* Upgrade sqlite-core to version 0.1.7 for Android
Upgrade powersync flutter libs to version 0.1.0

* Update demos versions
* Override default macOS sqlite db for tests

* Update to use flutter test rather than dart test

* Added documentation
* Check if abort future is completed before completing it

* Bump powersync version
* Use updated sqlite_async package

* Update to CommonDatabase for sqlite_async

* Update open DB to be synchronous

* Upgrade sqlite_async.
Update changelog and bump versions

* Fix linting errors
…feature/django-todolist-demo

# Conflicts:
#	demos/supabase-todolist/pubspec.lock
* Add hasSynced and waitForFirstSync

* Export sqlite3_common

* Add example code for hasSynced
Bump version
…nect

Close http connections immediately on disconnect
* Ensure future is not completed

* Bump version

* Add changelog entry for issue 114.

* Iterate through stream instead of listening to it

* Fix changelog message

---------

Co-authored-by: Ralf Kistner <[email protected]>
@stevensJourney
Copy link
Contributor

stevensJourney commented Jul 16, 2024

Tested this on most platforms - things seem to work normally.

An observation in web: due to the sync status not being shared in multiple tabs the hasSynced helper does not currently work in multiple tabs.

testalpha.mp4

@mugikhan
Copy link
Contributor Author

Tested this on most platforms - things seem to work normally.

An observation in web: due to the sync status not being shared in multiple tabs the hasSynced helper does not currently work in multiple tabs.

testalpha.mp4

Do we want to note this as a known issue or should I look into fixing it for alpha?

@stevensJourney
Copy link
Contributor

Tested this on most platforms - things seem to work normally.
An observation in web: due to the sync status not being shared in multiple tabs the hasSynced helper does not currently work in multiple tabs.
testalpha.mp4

Do we want to note this as a known issue or should I look into fixing it for alpha?

This issue seems to affect master as well if the device is offline. I created a PR #124 for the fix. We can update it here (there will be conflicts) after merging.

Copy link
Contributor

@stevensJourney stevensJourney left a comment

Choose a reason for hiding this comment

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

LGTM

@mugikhan mugikhan marked this pull request as ready for review July 17, 2024 15:48
@mugikhan mugikhan merged commit 4a6fe88 into alpha-release Jul 17, 2024
4 checks passed
@mugikhan mugikhan deleted the chore/merge-master-alpha-release branch July 17, 2024 15:49
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.

7 participants