Add ability to remove unplaced units
Clear ECS as well as db
fix failling test
Clean up object notation
Standardize constraint inputs across the GUI.
Removing superfluous this.remove()
Address QA issues.
Test fixes.
Because MBs are important.
Made env calls to the ecs more resilient by making it always execute the ecs's callbackWrapper method
Update service and use env to remove

The service's aggregated unit status was not being updated, thus the databinding
not catching it.  Also, we now use the environment's remove_units method, as that
takes care of everything that was being done before without adding the new
method to the ECS.
Merge branch 'develop' into delete-unplaced
Add env back into serviceunit-token
Fix test around removing
Merge pull request #527 from makyo/delete-unplaced

Add ability to remove unplaced units
Fixed ecs tests so that they also test that missing callback functions are added dynamically
undocumented run
Merge pull request #528 from kadams54/sync-constraints-1341751

Standardized constraint inputs across the GUI.
Fixed typos in comments
Merge pull request #532 from hatched/remove-unit-1365177

Fixes issue causing units removed from the inspector to hang around in the ecs.

When we made api calls of the environment but didn't provide a callback it would skip the ecs wrapper callback causing the ecs and db to get out of sync and break the GUI.

Fixes: 
 - #1365177 Removing a unit from the inspector no longer causes the unit to persist in the ecs
 - #1364549 Unable to delete a service with no unplaced units.