Misc cleanup per PR feedback

netbox/circuits/forms/model_forms.py

@@ -303,11 +303,15 @@ class VirtualCircuitTerminationForm(NetBoxModelForm):
         selector=True,
         query_params={
             'kind': 'virtual',
+            'virtual_circuit_termination_id': 'null',
+        },
+        context={
+            'parent': 'device',
         }
     )
 
     fieldsets = (
-        FieldSet('virtual_circuit', 'role', 'description', 'tags', 'interface'),
+        FieldSet('virtual_circuit', 'role', 'interface', 'description', 'tags'),
     )
 
     class Meta:

netbox/circuits/models/virtual_circuits.py

@@ -156,3 +156,9 @@ class VirtualCircuitTermination(
             return self.virtual_circuit.terminations.filter(
                 role=VirtualCircuitTerminationRoleChoices.ROLE_HUB
             )
+
+    def clean(self):
+        super().clean()
+
+        if self.interface and not self.interface.is_virtual:
+            raise ValidationError("Virtual circuits may be terminated only to virtual interfaces.")

netbox/templates/circuits/virtualcircuittermination.html

@@ -20,6 +20,7 @@
   <div class="row">
     <div class="col col-md-6">
       <div class="card">
+        <h2 class="card-header">{% trans "Virtual Circuit Termination" %}</h2>
         <table class="table table-hover attr-table">
           <tr>
             <th scope="row">{% trans "Provider" %}</th>
@@ -44,10 +45,31 @@
         </table>
       </div>
       {% include 'inc/panels/tags.html' %}
+      {% include 'inc/panels/custom_fields.html' %}
       {% plugin_left_page object %}
     </div>
     <div class="col col-md-6">
-      {% include 'inc/panels/custom_fields.html' %}
+      <div class="card">
+        <h2 class="card-header">{% trans "Interface" %}</h2>
+        <table class="table table-hover attr-table">
+          <tr>
+            <th scope="row">{% trans "Device" %}</th>
+            <td>{{ object.interface.device|linkify }}</td>
+          </tr>
+          <tr>
+            <th scope="row">{% trans "Interface" %}</th>
+            <td>{{ object.interface|linkify }}</td>
+          </tr>
+          <tr>
+            <th scope="row">{% trans "Type" %}</th>
+            <td>{{ object.interface.get_type_display }}</td>
+          </tr>
+          <tr>
+            <th scope="row">{% trans "Description" %}</th>
+            <td>{{ object.interface.description|placeholder }}</td>
+          </tr>
+        </table>
+      </div>
       {% plugin_right_page object %}
     </div>
   </div>