Skip to content
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

Firebase #17

Open
wants to merge 47 commits into
base: main
Choose a base branch
from
Open

Firebase #17

wants to merge 47 commits into from

Conversation

zgerbi
Copy link
Collaborator

@zgerbi zgerbi commented Apr 18, 2022

Implemented full sync functionality

zgerbi and others added 30 commits April 6, 2022 17:58
Tests and flutterfire integrated, tests are in Calendar tapdate
Sends all events to firebase with appropriate file structure
Also probably does todo lists too! But I can't check that yet
Needs more testing of multiple users and the to-do list, but all functionality *should* be implemented.

The method to use here is
Sync.sync(DateTime.now());
whenever you write to globals, and
Sync.sync(); to update either the local app or the database to the most recent changes. The latter should be called on startup, and maybe every minute or every few minutes.

Current problem logically is that there is the potential for a user to try to sync changes before they get the most up-to-date database, leading to data loss. Could maybe be solved with a listener on the database and syncing on any change.
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.

3 participants