Merge pull request #771 from uw-it-aca/task/zoom-minus-shared-netids

clear row when legacy netid revoked

Author: mikeseibel

endorsement/static/endorsement/js/endorse.js

@@ -111,7 +111,7 @@ var Endorse = (function () {
             _loadContainer();
             _registerEvents();
         },
-        
+
         endorse = function (modal_content_id, $rows) {
             var $modal = $('#endorse_modal'),
                 template = Handlebars.compile($('#' + modal_content_id).html()),
@@ -230,6 +230,11 @@ var Endorse = (function () {
                         return true;
                     }
 
+                    if (endorsement.endorsed === false && endorsement.is_legacy === true) {
+                        $row.remove();
+                        return true;
+                    }
+
                     is_first = $row.hasClass('endorsement_row_first');
                     is_even = $row.hasClass('endorsee_row_even');
 

endorsement/views/api/endorse.py

@@ -136,7 +136,19 @@ def _endorse(self, to_endorse, service, endorser, endorser_json,
                 try:
                     e = service.clear_endorsement(
                         endorser, endorsee)
-                    endorsements[service.service_name] = e.json_data()
+
+                    if (not endorsee.is_person
+                            and not service.supports_shared_netids):
+                        # legacy shared netid revoked
+                        endorsements[service.service_name] = {
+                            'endorser': endorser_json,
+                            'endorsee': endorsee.json_data(),
+                            'endorsed': False,
+                            'is_legacy': True
+                        }
+                    else:
+                        endorsements[service.service_name] = e.json_data()
+
                 except NoEndorsementException as ex:
                     endorsements[service.service_name] = {
                         'endorser': endorser_json,