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

Update master branch to use a localStorage persistence service #4

Open
leggetter opened this issue Feb 27, 2014 · 0 comments
Open

Update master branch to use a localStorage persistence service #4

leggetter opened this issue Feb 27, 2014 · 0 comments

Comments

@leggetter
Copy link
Contributor

At the moment the Items blade persists the data. This is naughty! Persistence should be performed via a service.

We should consider whether this service listens to/triggers events. Or whether the blades directly access it and it has specific Todo List domain methods.

The benefit of sticking with the EventHub solution is that it keeps the BRJS presentation simple (you don't need to show how to create and register a service).

The problems with this are that 'fire and forget' style messaging can lead to problems e.g. how does the persistence service know when to trigger the initial load of todo-added events? How does it know when the listening blades are ready?

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

No branches or pull requests

1 participant