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

Persist state. #38

Merged
merged 3 commits into from
Jan 18, 2017
Merged

Persist state. #38

merged 3 commits into from
Jan 18, 2017

Conversation

schuyler
Copy link
Owner

@schuyler schuyler commented Jan 17, 2017

Persist Redux store to AsyncStorage using redux-persist.

Implements a redux-persist "transform" filter that prevents persisting contact entries in the friend store which have never been activated in the app. This way we don't lug around the entire contact database; just the contacts the user cares enough to have clicked on at some point.

This PR also removes the scaffolded demo data used to initialize the store, prior to the previous merge.

At this point, we have an actual minimum viable app. It still needs a lot of love, but it actually would work in a rudimentary way for the intended purpose: You can pick people from your contacts, it keeps an ordered queue, you can mark people as contacted and it bumps them to the end of the queue, and it remembers your activity. HUZZAH! 🍾

@obra @natbat let us commemorate this moment, please

Implements a redux-persist "transform" filter that prevents persisting
contact entries in the friend store which have never been activated in
the app. This way we don't lug around the entire contact database; just
the contacts the user cares enough to have clicked on at some point.
@obra
Copy link
Collaborator

obra commented Jan 18, 2017

+1

@schuyler schuyler merged commit 994ebba into master Jan 18, 2017
@schuyler schuyler deleted the schuyler-persist-state branch January 18, 2017 01:34
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