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

[WIP] Add online and offline saving indicator #1283

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

myieye
Copy link
Collaborator

@myieye myieye commented Mar 27, 2022

I haven't touched this code for several months now and this PR is incomplete, but I just don't have much time for it right now.

I think the trickiest work is done. Perhaps I'll stay get around to providing a better description of what's missing and some comments in my code for the slightly complex streams/Observables.

For now, I just want to make sure the code doesn't get lost.


This change is Reviewable

@irahopkinson
Copy link
Collaborator

@myieye thanks for pushing this work for SF-1234 to the repo so it doesn't get lost.

@Nateowami Nateowami added the will require testing PR should not be merged until testers confirm testing is complete label Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
will require testing PR should not be merged until testers confirm testing is complete
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants