17178 review changes

netbox/templates/tenancy/contact.html

@@ -20,9 +20,15 @@
           <tr>
             <th scope="row">{% trans "Groups" %}</th>
             <td>
-              {% for group in object.groups.all %}
-                {{ group|linkify|placeholder }}{% if not forloop.last %}, {% endif %}
-              {% endfor %}
+              {% if object.groups.all|length > 0 %}
+                <ol class="list-unstyled mb-0">
+                  {% for group in object.groups.all %}
+                    <li>{{ group|linkify|placeholder }}</li>
+                  {% endfor %}
+                </ol>
+              {% else %}
+                {{ ''|placeholder }}
+              {% endif %}
             </td>
           </tr>
           <tr>

netbox/tenancy/models/contacts.py

@@ -13,6 +13,7 @@ __all__ = (
     'ContactAssignment',
     'Contact',
     'ContactGroup',
+    'ContactGroupMembership',
     'ContactRole',
 )
 
@@ -104,6 +105,8 @@ class ContactGroupMembership(models.Model):
         constraints = [
             models.UniqueConstraint(fields=['group', 'contact'], name='unique_group_name')
         ]
+        verbose_name = _('contact group membership')
+        verbose_name_plural = _('contact group memberships')
 
 
 class ContactAssignment(CustomFieldsMixin, ExportTemplatesMixin, TagsMixin, ChangeLoggedModel):