jforman / binder

Django Web Admin Gui for manging BIND DNS Zones
129 stars 41 forks source link

Refactors the views used to add and delete records. #42

Closed Dunedan closed 8 years ago

Dunedan commented 9 years ago

That pull request started as a simple one just trying to improve the implementation of the views a bit, but turned into a larger changeset improving the user experience in a few areas as noted below. The amount of changes might look scary at first sight, but isn't really that bad. Just try it out and enjoy the beauty of the new success and error visualizations.

Notable changes are:

jforman commented 8 years ago

I have made subsequent checkins to this branch to fix functionality that was broken as a part of your changes. Please test functionality in each pull request (I would highly suggest smaller ones, to make these massive reviews esaier on me).