move approvals to the new controller

app/controllers/approvals_controller.rb

@@ -0,0 +1,21 @@
+class ApprovalsController < ApplicationController
+  include MedicalReleaseContext
+
+  before_action :set_medical_release
+  before_action :set_project
+
+  layout "project"
+
+  def create
+    @medical_release.approve_by(current_user)
+    if @medical_release.save
+      redirect_to [@project, :medical_releases], notice: t('.release_approved')
+    end
+  end
+
+  private
+
+  def set_project
+    @project = @medical_release.project
+  end
+end

app/controllers/medical_releases_controller.rb

@@ -2,7 +2,7 @@ class MedicalReleasesController < ApplicationController
   include ProjectContext, MedicalReleaseContext
 
   before_action :set_project, only: [:index]
-  before_action :set_medical_release, only: [:destroy, :review, :approve]
+  before_action :set_medical_release, only: [:destroy]
 
   include ProjectLayout
 
@@ -18,19 +18,6 @@ class MedicalReleasesController < ApplicationController
     end
   end
 
-  def review
-    @project = @medical_release.project
-  end
-
-  def approve
-    @project = @medical_release.project
-
-    @medical_release.approve_by(current_user)
-    if @medical_release.save
-      redirect_to [@project, :medical_releases], notice: t('.release_approved')
-    end
-  end
-
   private
 
   def medical_releases

app/views/approvals/new.html.erb

@@ -3,7 +3,7 @@
   <div class="card-body">
     <embed class="embeded-contract-preview" type="application/pdf" src="<%= url_for [@medical_release, :contracts, format: "pdf"] %>" width="90%" height="1200" />
 
-    <%= bootstrap_form_with model: @medical_release, method: :patch, url: [:approve, @medical_release], local: true do |form| %>
+    <%= bootstrap_form_with model: @medical_release, method: :post, url: medical_release_approvals_path(@medical_release), local: true do |form| %>
       <div class="row align-items-center text-center mt-4">
         <%= link_to t("shared.cancel"), [@medical_release.project, :medical_releases], class: "col-3 text-reset" %>
         <div class="col-9">

app/views/medical_releases/_medical_release.html.erb

@@ -50,7 +50,7 @@
           <%= link_to fa_icon("tags fw", text: "Tags"), [:new, medical_release, :acts_as_taggable_on_tag], class: "dropdown-item", remote: true %>
         <% end %>
         <% if policy(MedicalRelease).review? %>
-          <%= link_to fa_icon("search fw", text: t('.actions.review')), review_medical_release_path(medical_release), class: "dropdown-item" %>
+          <%= link_to fa_icon("search fw", text: t('.actions.review')), new_medical_release_approvals_path(medical_release), class: "dropdown-item" %>
         <% end %>
         <% if policy(MedicalRelease).download_single? && policy(Contract).show? && (medical_release.contract.attached? || medical_release.contract_template.present?) %>
           <%= link_to fa_icon("download fw", text: "Download"), [medical_release, :contracts, format: "pdf"], class: "dropdown-item", target: "_blank" %>