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

Create Pouch Service #172

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

hamzahio
Copy link

@hamzahio hamzahio commented Feb 2, 2017

Good day Nolanlawson,

This pull request for creating service for the initiated db and we can access it from anywhere.

Benefits:
Authentication access current db and add authentication code.

stop syncing or anything else.

@akatov
Copy link
Contributor

akatov commented May 26, 2017

This looks too restrictive. As a consumer I am managing several pouches, e.g. one local, one remote, sometimes a local memory-only pouch. I roll my own service that manages these pouches and updates the adapter's db as and when necessary.
@hamzahio I don't think something like this should be part of ember-pouch, as it's too restrictive...

@broerse
Copy link
Collaborator

broerse commented May 26, 2017

As shown by https://bloggr.exmer.com/ you need something like this and not sync to get it working. I agree with @akatov that we don't want restrictive code in ember-pouch. I need to write documentation but authentication can be done with https://github.com/martinic/ember-simple-auth-pouch

Perhaps it is better to create ember-pouch-service but it still needs to be more flexible to support for instance the scenario above.

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