Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

LocalStorage #24

Merged
merged 4 commits into from
Mar 9, 2016
Merged

LocalStorage #24

merged 4 commits into from
Mar 9, 2016

Conversation

willdurand
Copy link
Member

  • Add localforage
  • Add loader
  • Tests (of course)

@willdurand willdurand changed the title [WIP] LocalStorage LocalStorage Mar 9, 2016
@willdurand
Copy link
Member Author

@jmaupetit it's real!

@willdurand willdurand changed the title LocalStorage [WIP] LocalStorage Mar 9, 2016
};
});

this.props.onSave(newRaw);
Copy link
Member

Choose a reason for hiding this comment

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

So you save every time the user changes the input raw content?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes but it's fast enough.

Copy link
Member

Choose a reason for hiding this comment

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

If you say so.

@willdurand willdurand changed the title [WIP] LocalStorage LocalStorage Mar 9, 2016
@willdurand
Copy link
Member Author

@jmaupetit I think I am ready

@jmaupetit
Copy link
Member

👍

jmaupetit added a commit that referenced this pull request Mar 9, 2016
@jmaupetit jmaupetit merged commit 35cb6e1 into master Mar 9, 2016
@jmaupetit jmaupetit deleted the localstorage branch March 9, 2016 22:15
willdurand pushed a commit that referenced this pull request Apr 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants