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

Install serialiser by default? #117

Open
backspace opened this issue Mar 5, 2016 · 1 comment
Open

Install serialiser by default? #117

backspace opened this issue Mar 5, 2016 · 1 comment

Comments

@backspace
Copy link
Collaborator

The _shouldSerializeHasMany override is good for saving child IDs in a parent, but it’s only enabled if someone chooses to install the ember-pouch serialiser. A new user encountered problems with this in #116. I see two solutions:

  1. Add a generator to create the serialiser file and list it alongside the adapter in the README
  2. Add the generator to the default blueprint so it’s installed with the addon

Maybe when the addon is installed, the adapter should also be created by default? Questions. Thoughts?

@mattmarcum
Copy link
Contributor

this #124 fixes #1

right now the default blueprint doesn't run any generators - it just installs pouch/relational-pouch via bower: https://github.com/nolanlawson/ember-pouch/blob/master/blueprints/ember-pouch/index.js

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

2 participants