-
Notifications
You must be signed in to change notification settings - Fork 45
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
Changes in model now reflect on form: Issue #82 & Issue #23 #92
Conversation
@asaf, will this be merged soon? I am debating forking from this branch as this change is make or break. |
+1 for this. will this be merged soon? |
@joshpfosi Have you created a fork for this feature? I'd be happy to help. |
I actually have not. I don't have time at the moment, but it should be straightforward to take this pull request, right? |
@joshpfosi for myself I just set my package.json to get this fork and commit for the time being, until this gets updated. I should probably fork @xcskier56 so I am not depending on his fork. |
Any progress on this? Cans someone list their package.json? How can I pull in |
Merged. |
I still experience this problem even with EmberForms 0.6.0. Looking into this now, but it appears to be the identical problem. |
@joshpfosi I'm not sure if this is not already fixed on this fork: https://github.com/piceaTech/ember-rapid-forms |
@pedrokiefer, I experience an issue when using ember-rapid-forms:
Any ideas on this? |
@joshpfosi never saw this error before. |
Hmm. Just created a fresh ember app and experienced the same issue: https://github.com/joshpfosi/ember-rapid-forms-bug Do you have a working Edit: Confirmed, without |
@joshpfosi I think it's something with the version of your ember-cli, but I might be mistaken. Confirmed, you are running ember 1.8, try updating to ember-cli 0.2.3 and ember 1.10 or 1.11. |
Changes by @BenjaminHorn and @michetti to allow updates in the model to be reflected in the form. This should close Issue #82 and Issue #23