search

jrief / django-angular

Let AngularJS play well with Django
http://django-angular.awesto.com/
MIT License
1.23k stars 294 forks source link

Fix #241, update build directory in Gruntfile #242

Closed mathead closed 8 years ago

adrienbrunet commented 8 years ago

You have merge conflicts. I did not want to take the time to rebase your branch to have the chance to test it (it's a 3lines update). I submitted another PR with your exact same change.

I tested it against my workflow, it works as expected but I'm not using your workflow which I still discourage. If you find it too disturbing, would you like to help us write documentation on how to properly add an update workflow?

Cheers and thanks for your PR.

mathead commented 8 years ago

Thanks for that! Not sure what you mean by documenting update workflow, but I could write something up.

adrienbrunet commented 8 years ago

I meant documenting the right way to make an "update view" i.e. how to properly bind your djng form with data you have in your controller. This is something we've been asked for... (see #184 ) And this would avoid some troubles you had previously for future users.

Kill two :bird: with one stone