Remove edit view for ConfigRevision instances

netbox/extras/tables/tables.py

@@ -43,7 +43,7 @@ REVISION_BUTTONS = """
 class ConfigRevisionTable(NetBoxTable):
     is_active = columns.BooleanColumn()
     actions = columns.ActionsColumn(
-        actions=('edit', 'delete'),
+        actions=('delete',),
         extra_buttons=REVISION_BUTTONS
     )
 

netbox/extras/urls.py

@@ -85,6 +85,13 @@ urlpatterns = [
     path('journal-entries/import/', views.JournalEntryBulkImportView.as_view(), name='journalentry_import'),
     path('journal-entries/<int:pk>/', include(get_model_urls('extras', 'journalentry'))),
 
+    # Config revisions
+    path('config-revisions/', views.ConfigRevisionListView.as_view(), name='configrevision_list'),
+    path('config-revisions/add/', views.ConfigRevisionEditView.as_view(), name='configrevision_add'),
+    path('config-revisions/delete/', views.ConfigRevisionBulkDeleteView.as_view(), name='configrevision_bulk_delete'),
+    path('config-revisions/<int:pk>/restore/', views.ConfigRevisionRestoreView.as_view(), name='configrevision_restore'),
+    path('config-revisions/<int:pk>/', include(get_model_urls('extras', 'configrevision'))),
+
     # Change logging
     path('changelog/', views.ObjectChangeListView.as_view(), name='objectchange_list'),
     path('changelog/<int:pk>/', include(get_model_urls('extras', 'objectchange'))),
@@ -115,11 +122,4 @@ urlpatterns = [
 
     # Markdown
     path('render/markdown/', views.RenderMarkdownView.as_view(), name="render_markdown"),
-
-    # Config Revision
-    path('config-revision/', views.ConfigRevisionListView.as_view(), name='configrevision_list'),
-    path('config-revision/add/', views.ConfigRevisionEditView.as_view(), name='configrevision_add'),
-    path('config-revision/delete/', views.ConfigRevisionBulkDeleteView.as_view(), name='configrevision_bulk_delete'),
-    path('config-revision/<int:pk>/restore', views.ConfigRevisionRestoreView.as_view(), name='configrevision_restore'),
-    path('config-revision/<int:pk>/', include(get_model_urls('extras', 'configrevision'))),
 ]

netbox/extras/views.py

@@ -1178,25 +1178,9 @@ class ScriptResultView(ContentTypePermissionRequiredMixin, View):
 
 
 #
-# Markdown
+# Config Revisions
 #
 
-class RenderMarkdownView(View):
-
-    def post(self, request):
-        form = forms.RenderMarkdownForm(request.POST)
-        if not form.is_valid():
-            HttpResponseBadRequest()
-        rendered = render_markdown(form.cleaned_data['text'])
-
-        return HttpResponse(rendered)
-
-
-#
-# Config Revision
-#
-
-
 class ConfigRevisionListView(generic.ObjectListView):
     queryset = ConfigRevision.objects.all()
     filterset = filtersets.ConfigRevisionFilterSet
@@ -1209,7 +1193,6 @@ class ConfigRevisionView(generic.ObjectView):
     queryset = ConfigRevision.objects.all()
 
 
-@register_model_view(ConfigRevision, 'edit')
 class ConfigRevisionEditView(generic.ObjectEditView):
     queryset = ConfigRevision.objects.all()
     form = forms.ConfigRevisionForm
@@ -1260,3 +1243,18 @@ class ConfigRevisionRestoreView(ContentTypePermissionRequiredMixin, View):
         messages.success(request, f"Restored configuration revision #{pk}")
 
         return redirect(reverse('extras:configrevision'))
+
+
+#
+# Markdown
+#
+
+class RenderMarkdownView(View):
+
+    def post(self, request):
+        form = forms.RenderMarkdownForm(request.POST)
+        if not form.is_valid():
+            HttpResponseBadRequest()
+        rendered = render_markdown(form.cleaned_data['text'])
+
+        return HttpResponse(rendered)

netbox/templates/extras/configrevision.html

@@ -10,18 +10,9 @@
 {% endblock %}
 
 {% block controls %}
-  {# Clone/Edit/Delete Buttons #}
   <div class="controls">
     <div class="control-group">
       {% plugin_buttons object %}
-
-      {# Extra buttons #}
-      {% block extra_controls %}{% endblock %}
-
-      {% if request.user|can_change:object %}
-        {% edit_button object %}
-      {% endif %}
-
     </div>
     <div class="control-group">
       {% custom_links object %}
@@ -29,10 +20,6 @@
   </div>
 {% endblock controls %}
 
-
-{% block tabs %}
-{% endblock %}
-
 {% block content %}
   <div class="row">
     <div class="col col-md-6">