manually report exception to sentry

app/controllers/api/medical_releases_controller.rb

@@ -1,11 +0,0 @@
-class Api::MedicalReleasesController < Api::ReleasesController
-  deserializable_resource :medical_release, only: [:create, :update]
-
-  def model_name
-    "medical_release"
-  end
-
-  def attributes_for_index
-    [:name]
-  end
-end

app/controllers/api/misc_releases_controller.rb

@@ -1,11 +0,0 @@
-class Api::MiscReleasesController < Api::ReleasesController
-  deserializable_resource :misc_release, only: [:create, :update]
-
-  def model_name
-    "misc_release"
-  end
-
-  def attributes_for_index
-    [:name]
-  end
-end

app/controllers/api/sync_controller.rb

@@ -10,10 +10,8 @@ class Api::SyncController < Api::ApiController
     @appearance_releases = (AppearanceRelease.where(project: accessible_projects))
     @location_releases = (LocationRelease.where(project: accessible_projects))
     @material_releases = (MaterialRelease.where(project: accessible_projects))
-    @medical_releases = MedicalRelease.where(project: accessible_projects)
-    @misc_releases = MiscRelease.where(project: accessible_projects)
     @talent_releases = (TalentRelease.where(project: accessible_projects))
-    @notes = notes_query(Note.where(notable: @appearance_releases + @location_releases + @material_releases + @medical_releases + @misc_releases + @talent_releases + @acquired_media_releases ))
+    @notes = notes_query(Note.where(notable: @appearance_releases + @location_releases + @material_releases + @talent_releases + @acquired_media_releases ))
 
     render json: {
       data: {
@@ -24,8 +22,6 @@ class Api::SyncController < Api::ApiController
         appearance_releases: releases_query(@appearance_releases),
         location_releases: releases_query(@location_releases),
         material_releases: releases_query(@material_releases),
-        medical_releases: releases_query(@medical_releases),
-        misc_releases: releases_query(@misc_releases),
         talent_releases: releases_query(@talent_releases),
         notes: @notes
       }

app/controllers/approvals_controller.rb

@@ -1,21 +0,0 @@
-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/contract_templates_controller.rb

@@ -62,19 +62,18 @@ class ContractTemplatesController < ApplicationController
   def contract_template_params
     params
       .require(:contract_template)
-      .permit(:name, :release_type, :body, :guardian_clause,
-              :signature_legal_text, :fee,
+      .permit(:name, :release_type, :body, :guardian_clause, :fee,
               :applicable_medium_id, :applicable_medium_text,
               :territory_id, :territory_text,
               :term_id, :term_text,
               :restriction_id, :restriction_text,
-              :question_1_text, :question_2_text,
-              :question_3_text, :question_4_text,
-              :question_5_text, :question_6_text,
-              :question_7_text, :question_8_text,
+              :question_1_text, :question_2_text, 
+              :question_3_text, :question_4_text, 
+              :question_5_text, :question_6_text, 
+              :question_7_text, :question_8_text, 
               :question_9_text, :question_10_text,
-              :question_11_text, :question_12_text,
-              :question_13_text, :question_14_text,
+              :question_11_text, :question_12_text, 
+              :question_13_text, :question_14_text, 
               :question_15_text)
   end
 

app/helpers/tooltip_helper.rb

@@ -5,12 +5,4 @@ module TooltipHelper
       concat tag.div(class: "tooltip-inner")
     end
   end
-
-  def get_approval_data_for_medical_release(medical_release)
-    if medical_release.approved_by_user_name.present?
-      "#{medical_release.approved_by_user_name} [#{medical_release.approved_by_user_email}]"
-    else
-      medical_release.approved_by_user_email
-    end
-  end
 end

app/jobs/generate_contracts_zip_job.rb

@@ -6,8 +6,8 @@ class GenerateContractsZipJob < ApplicationJob
   before_perform do |job|
     @project = job.arguments.first
     @download = job.arguments.second
-    @release_type = job.arguments.third
-    @folder_name = "#{@project.name.parameterize}_#{get_release_name(@release_type).gsub('_', '-')}"
+    release_type = job.arguments.third
+    @folder_name = "#{@project.name.parameterize}_#{get_release_name(release_type).gsub('_', '-')}"
     @download.update!(name: @folder_name, status: :pending)
   end
 
@@ -20,14 +20,9 @@ class GenerateContractsZipJob < ApplicationJob
         files.each do |attachment|
           zipfile.add(attachment, File.join("#{dir}/", attachment))
         end
-
-        if @release_type.constantize.include?(CsvExportable)
-          csv_file = generate_csv releases
-          zipfile.get_output_stream("#{@folder_name}.csv") { |f| f.puts(csv_file) }
-        end
       end
 
-      @download.file.attach(io: File.open(zipfile_name), filename: "#{@folder_name}.zip")
+      @download.file.attach(io: File.open(zipfile_name), filename: @folder_name)
     end
   rescue StandardError => e
     Rails.logger.error("Failed to generate download for project (##{project.id}) with release type #{release_type}\n" + e.message)
@@ -48,19 +43,6 @@ class GenerateContractsZipJob < ApplicationJob
 
   private
 
-  def generate_csv(releases)
-    release_class = @release_type.constantize
-    headers = release_class.csv_headers
-
-    CSV.generate(headers: true) do |csv|
-      csv << headers
-      releases.each do |release|
-        csv_row_data = release.to_csv_row
-        csv << csv_row_data
-      end
-    end
-  end
-
   def get_release_name(release_type)
     release_type.constantize.model_name.plural
   end

app/models/acquired_media_release.rb

@@ -9,13 +9,6 @@ class AcquiredMediaRelease < ApplicationRecord
   include Signable
   include Syncable
   include PersonName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name file_infos_count]
-    end
-  end
 
   has_many :file_infos, as: :releasable, dependent: :destroy
 
@@ -64,8 +57,4 @@ class AcquiredMediaRelease < ApplicationRecord
   def uses_edl?
     true
   end
-
-  def file_infos_count
-    file_infos.any? ? file_infos.size : I18n.t('acquired_media_releases.acquired_media_release.no_media')
-  end
 end

app/models/appearance_release.rb

@@ -15,13 +15,6 @@ class AppearanceRelease < ApplicationRecord
   include SecondGuardianPhotoable
   include GuardianName
   include SecondGuardianName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name contact_info]
-    end
-  end
 
   has_one_attached :person_photo
 

app/models/concerns/csv_exportable.rb

@@ -1,56 +0,0 @@
-# frozen_string_literal: true
-
-module CsvExportable
-  extend ActiveSupport::Concern
-
-  COMMON_HEADERS = %i[notes tags signed_at].freeze
-  COMMON_VALUES = %w[clean_notes clean_tags signed_on].freeze
-
-  included do
-    class << self
-      def custom_csv_exportable_headers
-        []
-      end
-
-      def csv_headers
-        headers = custom_csv_exportable_headers + COMMON_HEADERS
-
-        headers.map do |header|
-          I18n.t("#{model_name.plural}.index.table_headers.#{header}")
-        end
-      end
-    end
-
-    def to_csv_row
-      (self.class.custom_csv_exportable_headers + COMMON_VALUES).map do |function|
-        send(function)
-      end
-    end
-
-    private
-
-    def contact_info
-      contact_info = ''
-      contact_info += "#{person_address}; " if person_address.present?
-      contact_info += "P: #{person_phone}; " if person_phone.present?
-      contact_info += "E: #{person_email}" if person_email.present?
-      contact_info.delete_suffix '; '
-    end
-
-    def clean_notes
-      notes = ''
-      self.notes.order_by_recent.each do |note|
-        notes += "#{note.content}(#{note.email}), "
-      end
-      notes.delete_suffix ', '
-    end
-
-    def clean_tags
-      tags = ''
-      self.tags.each do |tag|
-        tags += "#{tag.name}, "
-      end
-      tags.delete_suffix ', '
-    end
-  end
-end

app/models/contract_template.rb

@@ -21,7 +21,6 @@ class ContractTemplate < ApplicationRecord
   monetize :fee_cents
   has_rich_text :body
   has_rich_text :guardian_clause
-  has_rich_text :signature_legal_text
 
   validates :name, presence: true
   validates :release_type, presence: true

app/models/location_release.rb

@@ -10,13 +10,6 @@ class LocationRelease < ApplicationRecord
   include Syncable
   include Taggable
   include PersonName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name address]
-    end
-  end
 
   composed_of :address,
               mapping: [

app/models/material_release.rb

@@ -10,15 +10,8 @@ class MaterialRelease < ApplicationRecord
   include Syncable
   include Taggable
   include PersonName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name]
-    end
-  end
-
-  composed_of :person_address,
+  
+  composed_of :person_address, 
               class_name: "Address",
               mapping: [
                 %w(person_address_street1 street1),
@@ -37,15 +30,15 @@ class MaterialRelease < ApplicationRecord
     validates :signature, attached: true
   end
 
-  searchable_on %i[
-    name
+  searchable_on %i[ 
+    name 
     person_address_street1 person_address_street2 person_address_city person_address_state person_address_zip person_address_country
   ]
 
   def contact_person
     @contact_person ||= Contact.new(person_name, person_address, person_email, person_phone)
   end
-
+  
   def minor?
     false
   end

app/models/medical_release.rb

@@ -11,13 +11,6 @@ class MedicalRelease < ApplicationRecord
   include SecondGuardianPhotoable
   include GuardianName
   include SecondGuardianName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[approved? name contact_info]
-    end
-  end
 
   NUMBER_OF_CUSTOM_FIELDS = 15
 
@@ -107,18 +100,6 @@ class MedicalRelease < ApplicationRecord
     "#{project.name.parameterize}_#{contract_template.release_type}_#{(signed_at || created_at).strftime("%Y.%m.%d")}_#{release_number}_#{filename_suffix.parameterize}"
   end
 
-  def approve_by(user)
-    return unless approved_at.nil?
-
-    self.approved_by_user_name = user.full_name
-    self.approved_by_user_email = user.email
-    self.approved_at = Time.zone.now
-  end
-
-  def approved?
-    approved_at.present?
-  end
-
   private
 
   def valid_answers

app/models/misc_release.rb

@@ -9,13 +9,6 @@ class MiscRelease < ApplicationRecord
   include PersonName
   include GuardianName
   include GuardianPhotoable
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name contact_info]
-    end
-  end
 
   NUMBER_OF_CUSTOM_FIELDS = 15
 

app/models/music_release.rb

@@ -7,14 +7,7 @@ class MusicRelease < ApplicationRecord
   include Searchable
   include Taggable
   include PersonName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name file_infos_count composers_count publishers_count]
-    end
-  end
-
+  
   has_many :file_infos, as: :releasable, dependent: :destroy
   has_many :composers, dependent: :destroy
   has_many :publishers, dependent: :destroy
@@ -79,18 +72,6 @@ class MusicRelease < ApplicationRecord
     false
   end
 
-  def file_infos_count
-    file_infos.size
-  end
-
-  def composers_count
-    composers.size
-  end
-
-  def publishers_count
-    publishers.size
-  end
-
   private
 
   def publisher_percentages_add_up_to_100

app/models/qr_code.rb

@@ -10,7 +10,7 @@ class QrCode
     url       = Rails.application.routes.url_helpers.url_for(route)
     filename  = [contract_template.project.name, contract_template.name].map(&:parameterize).join("_")
 
-    new(url, "#{filename}.png")
+    new(url, filename)
   end
 
   def initialize(url, filename = "qrcode.png")

app/models/talent_release.rb

@@ -14,13 +14,6 @@ class TalentRelease < ApplicationRecord
   include SecondGuardianPhotoable
   include GuardianName
   include SecondGuardianName
-  include CsvExportable
-
-  class << self
-    def custom_csv_exportable_headers
-      %i[name phone email]
-    end
-  end
 
   composed_of :person_address, 
               class_name: "Address",
@@ -93,14 +86,6 @@ class TalentRelease < ApplicationRecord
     person_name
   end
 
-  def phone
-    person_phone
-  end
-
-  def email
-    person_email
-  end
-
   def filename_suffix
     "#{person_last_name} #{person_first_name}"
   end

app/policies/medical_release_policy.rb

@@ -15,14 +15,6 @@ class MedicalReleasePolicy < ReleasePolicy
     user.manager? || user.account_manager?
   end
 
-  def review?
-    user.account_manager?
-  end
-
-  def approve?
-    review?
-  end
-
   def edit_photos?
     true
   end

app/views/approvals/new.html.erb

@@ -1,15 +0,0 @@
-<div class="card shadow-sm">
-  <%= card_header text: t(".heading"), close_action_path: [@project, :medical_releases] %>
-  <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: :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">
-          <%= form.button t('.actions.approve'), id: "approve_release", class: class_string("btn btn-block btn-success btn-primary"), data: { disable_with: t("shared.disable_with") } %>
-        </div>
-      </div>
-    <% end %>
-  </div>
-</div>

app/views/broadcasts/splash.html.erb

@@ -18,7 +18,15 @@
       <div class="card-body p-0">
         <div class="embed-responsive embed-responsive-16by9">
           <div class="embed-responsive-item">
-            <div style="padding:56.25% 0 0 0;position:relative;"><iframe src="https://player.vimeo.com/video/435943632" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="autoplay; fullscreen" allowfullscreen></iframe></div><script src="https://player.vimeo.com/api/player.js"></script>
+            <table class="w-100 h-100 bg-secondary">
+              <tbody>
+              <tr>
+                <td class="text-center align-middle text-white">
+                  Video tutorial will be available soon
+                </td>
+              </tr>
+              </tbody>
+            </table>
           </div>
         </div>
       </div>

app/views/contract_templates/_contract_template.html.erb

@@ -15,7 +15,7 @@
   <td>
     <%= contract_template.releases.size %>
   </td>
-  <td class="text-right" nowrap>
+  <td class="text-right">
     <div class="btn-group">
       <%= button_tag t(".actions.manage"), class: "btn btn-light btn-sm dropdown-toggle border", data: { toggle: "dropdown", boundary: "window" }, aria: { haspopup: true, expanded: false } %>
       <div class="dropdown-menu dropdown-menu-right">
@@ -27,7 +27,7 @@
           <%= link_to fa_icon("print", text: "Print"), [:new, contract_template, :blank_contracts], class: "dropdown-item", target: :_blank %>
         <% end %>
         <% if policy(contract_template).destroy? %>
-          <%= link_to fa_icon("trash", text: "Delete"), contract_template, class: "dropdown-item", method: :delete, data: { confirm: "Are you sure?" } %>
+          <%= link_to fa_icon("archive", text: "Archive"), contract_template, class: "dropdown-item", method: :delete, data: { confirm: "Are you sure?" } %>
         <% end %>
       </div>
     </div>

app/views/contract_templates/_form.html.erb

@@ -25,11 +25,6 @@
         <%= form.rich_text_area :guardian_clause %>
       <% end %>
     </div>
-    <div id="signature_legal_text">
-      <%= form.form_group do %>
-        <%= form.rich_text_area :signature_legal_text %>
-      <% end %>
-    </div>
   <% end %>
 
   <%= field_set_tag content_tag(:span, t(".custom_fields.heading"), class: "h6 text-muted text-uppercase"), id: "custom_fields", style: "display: none;" do %>

app/views/contract_templates/splash.html.erb

@@ -18,7 +18,15 @@
       <div class="card-body p-0">
         <div class="embed-responsive embed-responsive-16by9">
           <div class="embed-responsive-item">
-            <div style="padding:56.25% 0 0 0;position:relative;"><iframe src="https://player.vimeo.com/video/435942851" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="autoplay; fullscreen" allowfullscreen></iframe></div><script src="https://player.vimeo.com/api/player.js"></script>
+            <table class="w-100 h-100 bg-secondary">
+              <tbody>
+              <tr>
+                <td class="text-center align-middle text-white">
+                  Video tutorial will be available soon
+                </td>
+              </tr>
+              </tbody>
+            </table>
           </div>
         </div>
       </div>

app/views/contracts/_for_office_use_only.erb

@@ -1,13 +0,0 @@
-<% if preview %>
-  <h1>PREVIEW ONLY</h1>
-<% end %>
-
-<p class="heading"><strong><u><%= t '.heading' %></u></strong></p>
-
-<dl>
-  <%= description_list_pair t('.description_labels.producer'), releasable.project.account.name %>
-  <%= description_list_pair t('.description_labels.production'), releasable.project.name %>
-  <%= description_list_pair t('.description_labels.employee_issued_to'), releasable.name %>
-  <%= description_list_pair t('.description_labels.issued_by'), releasable.approved_by_user_name %>
-  <%= description_list_pair t('.description_labels.date_issued'), releasable.approved_at %>
-</dl>

app/views/contracts/pdf.html.erb

@@ -25,13 +25,6 @@
 <div class="page">
   <%= render "contracts/signature_page", releasable: releasable, contract_template: contract_template, preview: preview %>
 </div>
-
-<% if releasable.class == MedicalRelease && releasable.approved? %>
-  <div class="page">
-    <%= render "contracts/for_office_use_only", releasable: releasable, preview: preview %>
-  </div>
-<% end %>
-
 <% if releasable.class == AcquiredMediaRelease %>
   <div class="page">
     <%= render "contracts/files", release: releasable, preview: preview %>

app/views/medical_releases/_medical_release.html.erb

@@ -1,13 +1,5 @@
 <tr id="<%= dom_id(medical_release) %>">
   <td data-behavior="select"><%= check_box_tag "medical_release_ids[]", medical_release.id, false %></td>
-  <td class="text-center">
-    <% if medical_release.approved? %>
-      <% tooltip_user_data = get_approval_data_for_medical_release(medical_release) %>
-      <i class="fa fa-check-circle fa-2x text-success"
-         data-toggle="tooltip"
-         title="<%= t '.messages.approved_tooltip', user: tooltip_user_data, timestamp: medical_release.approved_at %>"></i>
-    <% end %>
-  </td>
   <td>
     <% if medical_release.photo.attached? %>
       <%= image_tag medium_variant(medical_release.photo), class: "img-fluid" %>
@@ -45,9 +37,6 @@
         <% if policy(medical_release.tags).new? %>
           <%= 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')), 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" %>
         <% end %>

app/views/medical_releases/index.html.erb

@@ -22,7 +22,6 @@
     <thead class="thead-light">
       <tr>
         <th data-behavior="all-selectable"><%= check_box_tag "medical_release_ids[]", false, false %></th>
-        <th><%= t '.table_headers.approved'%></th>
         <th></th>
         <th><%= MedicalRelease.human_attribute_name(:person_name) %></th>
         <th><%= MedicalRelease.human_attribute_name(:contact_info) %></th>

app/views/medical_releases/review.html.erb

@@ -1,15 +0,0 @@
-<div class="card shadow-sm">
-  <%= card_header text: t(".heading"), close_action_path: [@project, :medical_releases] %>
-  <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| %>
-      <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">
-          <%= form.button t('.actions.approve'), id: "approve_release", class: class_string("btn btn-block btn-success btn-primary"), data: { disable_with: t("shared.disable_with") } %>
-        </div>
-      </div>
-    <% end %>
-  </div>
-</div>

app/views/public/acquired_media_releases/new.html.erb

@@ -50,7 +50,7 @@
       <hr>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form %>
       <% end %>
 
       <div class="mt-5">

app/views/public/appearance_releases/new.html.erb

@@ -156,7 +156,7 @@
       <% end %>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form %>
       <% end %>
 
       <div class="mt-5">

app/views/public/location_releases/new.html.erb

@@ -50,7 +50,7 @@
       <% end %>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, instruction: 'An Authorized Signatory', signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form, instruction: 'An Authorized Signatory' %>
       <% end %>
 
       <div class="mt-5">

app/views/public/material_releases/new.html.erb

@@ -41,7 +41,7 @@
       <hr>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, instruction: 'For Owner or Authorized Signatory', signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form, instruction: 'For Owner or Authorized Signatory' %>
       <% end %>
 
       <div class="mt-5">

app/views/public/medical_releases/new.html.erb

@@ -145,7 +145,7 @@
       <% end %>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form %>
       <% end %>
 
       <div class="mt-5">

app/views/public/misc_releases/new.html.erb

@@ -39,7 +39,9 @@
           <%= form.text_field :person_first_name, required: true, wrapper_class: "col-sm-6" %>
           <%= form.text_field :person_last_name, required: true, wrapper_class: "col-sm-6" %>
           <%= form.phone_field :person_phone, wrapper_class: "col-sm-6" %>
-          <%= form.email_field :person_email, required: true, wrapper_class: "col-sm-6" %>
+        </div>
+        <div class="form-row">
+          <%= form.email_field :person_email, wrapper_class: "col-sm-6" %>
         </div>
         <%= render "shared/address_fields", form: form, subject: "person" %>
       <% end %>
@@ -95,7 +97,7 @@
       <% end %>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form %>
       <% end %>
 
       <div class="mt-5">

app/views/public/talent_releases/new.html.erb

@@ -131,7 +131,7 @@
       <% end %>
 
       <%= card_field_set_tag t(".signature.heading") do %>
-        <%= render "shared/signature_fields", form: form, signature_legal_text: @contract_template.signature_legal_text %>
+        <%= render "shared/signature_fields", form: form %>
       <% end %>
 
       <div class="mt-5">

app/views/shared/_signature_fields.html.erb

@@ -12,8 +12,4 @@
     <%= fa_icon "refresh" %> <%= t "shared.clear" %>
   <% end %>
 </div>
-<br>
 
-<% if local_assigns[:signature_legal_text] && signature_legal_text.present? %>
-  <div class="alert alert-warning font-weight-bold"><%= signature_legal_text %></div>
-<% end %>

app/views/task_requests/create.html.erb

@@ -13,4 +13,3 @@
   <% end %>
 <% end %>
 <p class="alert alert-success p-3 lead text-center"><%= t '.success_message' %></p>
-<%= link_to fa_icon("arrow-left", text: "Back"), [@project, :task_requests], class: "btn btn-primary" %>

app/views/videos/splash.html.erb

@@ -6,7 +6,7 @@
 
   <%= link_to t(".actions.book_demo"), 'https://meetings.hubspot.com/bray2', class: "btn btn-primary border align-self-center h-50 ml-auto mr-2 pb-2", target: '_blank' %>
   <% if policy(Video).new? %>
-    <%= link_to t(".actions.upload_video"), [:landing, @project, :videos], class: "btn btn-success border align-self-center h-50 pb-2" %>
+    <%= link_to t(".actions.upload_video"), [:new, @project, :video], class: "btn btn-success border align-self-center h-50 pb-2" %>
   <% end %>
 </div>
 
@@ -18,7 +18,15 @@
       <div class="card-body p-0">
         <div class="embed-responsive embed-responsive-16by9">
           <div class="embed-responsive-item">
-            <div style="padding:56.25% 0 0 0;position:relative;"><iframe src="https://player.vimeo.com/video/435943995" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="autoplay; fullscreen" allowfullscreen></iframe></div><script src="https://player.vimeo.com/api/player.js"></script>
+            <table class="w-100 h-100 bg-secondary">
+              <tbody>
+              <tr>
+                <td class="text-center align-middle text-white">
+                  Video tutorial will be available soon
+                </td>
+              </tr>
+              </tbody>
+            </table>
           </div>
         </div>
       </div>

config/locales/en.yml

@@ -50,7 +50,6 @@ en:
       empty: Acquired Media Releases will appear here
       table_headers:
         file_infos_count: No. Files
-        name: Name
         notes: Notes
         signed_at: Date Signed
         tags: Tags
@@ -152,8 +151,6 @@ en:
       empty: Appearance Releases will appear here
       imported_appearance_release_missing_attachment: Person photo or contract missing for imported appearance release
       table_headers:
-        contact_info: Contact info
-        name: Name
         notes: Notes
         signed_at: Date Signed
         tags: Tags
@@ -174,13 +171,6 @@ en:
     sidebar:
       files: Files
       team_member: Team Member
-  approvals:
-    create:
-      release_approved: Medical release has been approved
-    new:
-      actions:
-        approve: Approve
-      heading: Review Medical Release
   blank_contracts:
     new:
       number_of_copies_label: Number of copies
@@ -298,14 +288,6 @@ en:
         print_QR_code: Print out release QR codes
         releases_automatically_organized: Releases are automatically organized as they’re submitted
   contracts:
-    for_office_use_only:
-      description_labels:
-        date_issued: Date Issued
-        employee_issued_to: Employee Issued To
-        issued_by: Issued By
-        producer: Producer
-        production: Production
-      heading: For Office Use Only
     photos:
       guardian_2_photo_heading: Second guardian photo
       guardian_photo_heading: Guardian photo
@@ -382,7 +364,6 @@ en:
       contract_template:
         fee: Leave at $0.00 for no-fee
         guardian_clause: Leave blank if not required for this contract
-        signature_legal_text: Leave blank if not required for this contract
       task_request:
         time_allowed: Minimum of 2 hours, no partial hours allowed
       video:
@@ -745,7 +726,6 @@ en:
       empty: Location Releases will appear here
       table_headers:
         address: Address
-        name: Name
         notes: Notes
         signed_at: Date Signed
         tags: Tags
@@ -780,7 +760,6 @@ en:
         search: Search
       empty: Material Releases will appear here
       table_headers:
-        name: Name
         notes: Notes
         signed_at: Date Signed
         tags: Tags
@@ -802,19 +781,12 @@ en:
         search: Search
       empty: Medical releases will appear here
       table_headers:
-        approved: Approved
-        approved?: Approved
-        contact_info: Contact info
-        name: Person name
         notes: Notes
         signed_at: Date Signed
         tags: Tags
     medical_release:
       actions:
         manage: Manage
-        review: Review
-      messages:
-        approved_tooltip: Approved by %{user} on %{timestamp}
   misc_releases:
     destroy:
       alert: The misc release has been deleted
@@ -823,8 +795,6 @@ en:
         search: Search
       empty: Misc Releases will appear here
       table_headers:
-        contact_info: Contact info
-        name: Person name
         notes: Notes
         signed_at: Date Signed
         tags: Tags
@@ -859,7 +829,6 @@ en:
       table_headers:
         composers_count: No. Composers
         file_infos_count: No. Files
-        name: Name
         notes: Notes
         publishers_count: No. Publishers
         signed_at: Date Signed
@@ -1256,10 +1225,7 @@ en:
         search: Search
       empty: Talent Releases will appear here
       table_headers:
-        email: Email
-        name: Name
         notes: Notes
-        phone: Phone
         signed_at: Date Signed
         tags: Tags
     new:

config/locales/es.yml

@@ -1,14 +1,4 @@
 es:
-  acquired_media_releases:
-    acquired_media_release:
-      no_media: No Media (ES)
-    index:
-      table_headers:
-        file_infos_count: No. Files (ES)
-        name: Name (ES)
-        notes: Notes (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
   activerecord:
     attributes:
       appearance_release:
@@ -51,12 +41,6 @@ es:
           heading: Person Photo (ES)
     index:
       imported_appearance_release_missing_attachment: Person photo or contract missing for imported appearance release (ES)
-      table_headers:
-        contact_info: ""
-        name: ""
-        notes: ""
-        signed_at: ""
-        tags: ""
     shared:
       imported_appearance_release_contract_name: Contrato Importado
       imported_appearance_release_headshot_name: Retrato Importado
@@ -66,13 +50,6 @@ es:
       all_releases: All Releases (ES)
       complete_releases: Complete Releases (ES)
       incomplete_releases: Incomplete Releases (ES)
-  approvals:
-    create:
-      release_approved: Medical release has been approved (ES)
-    new:
-      actions:
-        approve: Approve (ES)
-      heading: Review Medical Release (ES)
   blank_contracts:
     new:
       number_of_copies_label: Number of copies (ES)
@@ -130,14 +107,6 @@ es:
         print_QR_code: Print out release QR codes (ES)
         releases_automatically_organized: Releases are automatically organized as they’re submitted (ES)
   contracts:
-    for_office_use_only:
-      description_labels:
-        date_issued: Date Issued (ES)
-        employee_issued_to: Employee Issued To (ES)
-        issued_by: Issued By (ES)
-        producer: Producer (ES)
-        production: Production (ES)
-      heading: For Office Use Only (ES)
     photos:
       guardian_2_photo_heading: Second guardian photo (ES)
       guardian_photo_heading: Guardian photo (ES)
@@ -191,7 +160,6 @@ es:
       contract_template:
         fee: Leave at $0.00 for no-fee (ES)
         guardian_clause: Leave blank if not required for this contract (ES)
-        signature_legal_text: Leave blank if not required for this contract (ES)
     label:
       appearance_release:
         guardian_2_address_city: Guardian 2 city (ES)
@@ -305,64 +273,18 @@ es:
         create: Create Live Stream (ES)
         update: Save Changes (ES)
       create: 'Crear %{model}'
-      medical_release:
-        update: Approve (ES)
       update: 'Actualizar %{model}'
   location_releases:
     form:
       photos:
         dropzone_label: Tap to take a photo of the Property (optional) (ES)
-    index:
-      table_headers:
-        address: Address (ES)
-        notes: Notes (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
   material_releases:
     form:
       photos:
         dropzone_label: Tap to take a photo of Licensed Material (optional) (ES)
-    index:
-      table_headers:
-        name: Name (ES)
-        notes: Notes (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
   medical_releases:
     custom_validation_errors:
       question_answer_is_required: answer is required (ES)
-    index:
-      table_headers:
-        approved: Approved (ES)
-        approved?: Approved (ES)
-        contact_info: Contact info (ES)
-        name: Person name (ES)
-        notes: Notes (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
-    medical_release:
-      actions:
-        review: Review (ES)
-      messages:
-        approved_tooltip: ""
-  misc_releases:
-    index:
-      table_headers:
-        contact_info: Contact info (ES)
-        name: Person name (ES)
-        notes: Notes (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
-  music_releases:
-    index:
-      table_headers:
-        composers_count: No. Composers (ES)
-        file_infos_count: No. Files (ES)
-        name: Name (ES)
-        notes: Notes (ES)
-        publishers_count: No. Publishers (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
   public:
     appearance_releases:
       create:
@@ -467,14 +389,6 @@ es:
           heading: Second Guardian Photo (ES)
         guardian_photo:
           heading: Guardian Photo (ES)
-    index:
-      table_headers:
-        email: Email (ES)
-        name: Name (ES)
-        notes: Notes (ES)
-        phone: Phone (ES)
-        signed_at: Date Signed (ES)
-        tags: Tags (ES)
   task_requests:
     create:
       success_message: Your task request was successfully submitted. Thank you. A chat window will pop up on the lower right in a few seconds. (ES)

config/routes.rb

@@ -2,7 +2,7 @@ require 'oath/constraints/signed_in'
 require 'sidekiq/web'
 
 Rails.application.routes.draw do
-  AVAILABLE_LOCALES_REGEX = /#{I18n.available_locales.join("|")}/.freeze
+  AVAILABLE_LOCALES_REGEX = /#{I18n.available_locales.join("|")}/
 
   concern :confirmable do
     resources :video_release_confirmations, only: [:new, :create, :destroy]
@@ -136,18 +136,13 @@ Rails.application.routes.draw do
       end
     end
 
-    RELEASES = [:acquired_media_releases, :appearance_releases, :talent_releases, :material_releases, :medical_releases, :misc_releases, :location_releases].freeze
-    ALL_RELEASES = RELEASES + [:music_releases]
+    RELEASES = [:acquired_media_releases, :appearance_releases, :talent_releases, :material_releases, :location_releases]
+    ALL_RELEASES = RELEASES + [:music_releases, :medical_releases, :misc_releases]
 
     ALL_RELEASES.each do |release|
       resources release, only: [], concerns: :taggable
     end
 
-    # Customization for medical releases
-    resources :medical_releases, only: [], concerns: :taggable do
-      resource :approvals, only: [:new, :create]
-    end
-
     resources :bulk_taggings, only: [:new, :create]
 
     namespace :api do
@@ -163,7 +158,7 @@ Rails.application.routes.draw do
           resources :contract_templates, only: [:index]
         end
         resources :contract_templates, only: [:show] do
-          (RELEASES - [:misc_releases, :medical_releases]).each do |release|
+          RELEASES.each do |release|
             resources release, controller: release, only: [:create]
           end
         end

db/migrate/20200707155717_add_approval_info_columns_to_medical_releases.rb

@@ -1,7 +0,0 @@
-class AddApprovalInfoColumnsToMedicalReleases < ActiveRecord::Migration[6.0]
-  def change
-    add_column :medical_releases, :approved_by_user_name, :text
-    add_column :medical_releases, :approved_by_user_email, :text
-    add_column :medical_releases, :approved_at, :timestamp
-  end
-end

db/structure.sql

@@ -1045,10 +1045,7 @@ CREATE TABLE public.medical_releases (
     guardian_2_address_city character varying,
     guardian_2_address_state character varying,
     guardian_2_address_zip character varying,
-    guardian_2_address_country character varying,
-    approved_by_user_name text,
-    approved_by_user_email text,
-    approved_at timestamp without time zone
+    guardian_2_address_country character varying
 );
 
 
@@ -3905,7 +3902,6 @@ INSERT INTO "schema_migrations" (version) VALUES
 ('20200619134853'),
 ('20200622180507'),
 ('20200625144713'),
-('20200702152130'),
-('20200707155717');
+('20200702152130');
 
 

spec/controllers/api/medical_releases_controller_spec.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Api::MedicalReleasesController, type: :controller do
-  let(:current_user) { create(:user) }
-  let(:project) { create(:project, name: 'first', account: current_user.primary_account) }
-
-  describe '#index' do
-    it 'returns a succesful response' do
-      create(:medical_release, person_first_name: 'John', person_last_name: 'Lee', project_id: project.id)
-      create(:medical_release, person_first_name: 'Jane', person_last_name: 'Lee', project_id: project.id)
-
-      sign_in_to_api(current_user)
-      get :index, params: { project_id: project.id }
-
-      expect(response).to be_successful
-      expect(response.body).to include 'John'
-      expect(response.body).to include 'Jane'
-    end
-  end
-
-  describe '#show' do
-    it 'returns a succesful response' do
-      release1 = create(:medical_release, person_first_name: 'John', person_last_name: 'Lee', project_id: project.id)
-      release2 = create(:medical_release, person_first_name: 'Jane', person_last_name: 'Lee', project_id: project.id)
-
-      sign_in_to_api(current_user)
-      get :show, params: { id: release1 }
-
-      expect(response).to be_successful
-      expect(response.body).to include 'John'
-      expect(response.body).not_to include 'Jane'
-
-      get :show, params: { id: release2 }
-
-      expect(response).to be_successful
-      expect(response.body).not_to include 'John'
-      expect(response.body).to include 'Jane'
-    end
-  end
-end

spec/controllers/api/misc_releases_controller_spec.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Api::MiscReleasesController, type: :controller do
-  let(:current_user) { create(:user) }
-  let(:project) { create(:project, name: 'first', account: current_user.primary_account) }
-
-  describe '#index' do
-    it 'returns a succesful response' do
-      create(:misc_release, person_first_name: 'John', person_last_name: 'Lee', project_id: project.id)
-      create(:misc_release, person_first_name: 'Jane', person_last_name: 'Lee', project_id: project.id)
-
-      sign_in_to_api(current_user)
-      get :index, params: { project_id: project.id }
-
-      expect(response).to be_successful
-      expect(response.body).to include 'John'
-      expect(response.body).to include 'Jane'
-    end
-  end
-
-  describe '#show' do
-    it 'returns a succesful response' do
-      release1 = create(:misc_release, person_first_name: 'John', person_last_name: 'Lee', project_id: project.id)
-      release2 = create(:misc_release, person_first_name: 'Jane', person_last_name: 'Lee', project_id: project.id)
-
-      sign_in_to_api(current_user)
-      get :show, params: { id: release1 }
-
-      expect(response).to be_successful
-      expect(response.body).to include 'John'
-      expect(response.body).not_to include 'Jane'
-
-      get :show, params: { id: release2 }
-
-      expect(response).to be_successful
-      expect(response.body).not_to include 'John'
-      expect(response.body).to include 'Jane'
-    end
-  end
-end

spec/controllers/api/notes_controller_spec.rb

@@ -18,14 +18,6 @@ releases = [
   {
     type: :material_release,
     obligatory_attribute: :name
-  },
-  {
-    type: :medical_release,
-    obligatory_attribute: :person_name
-  },
-  {
-    type: :misc_release,
-    obligatory_attribute: :person_name
   }
 ]
 

spec/controllers/api/sync_controller_spec.rb

@@ -110,26 +110,6 @@ RSpec.describe Api::SyncController, type: :controller do
       expect(guardian_photo).to include('id', 'type', 'attributes')
       expect(photo_attributes).to include('filename', 'content_type', 'url', 'thumbnail_url')
     end
-
-    it 'contains misc releases' do
-      create_default_data
-
-      get :index
-
-      misc_releases = attributes_for_type('misc_releases')
-
-      expect(misc_releases.first).to include('id')
-    end
-
-    it 'contains misc releases' do
-      create_default_data
-
-      get :index
-
-      medical_releases = attributes_for_type('medical_releases')
-
-      expect(medical_releases.first).to include('id')
-    end
   end
 
   private
@@ -140,8 +120,6 @@ RSpec.describe Api::SyncController, type: :controller do
     create(:talent_release, project: project)
     create(:location_release, project: project)
     create(:material_release, project: project)
-    create(:medical_release, project: project)
-    create(:misc_release, project: project)
   end
 
   def create_default_data_with_guardian_info

spec/controllers/approvals_controller_spec.rb

@@ -1,36 +0,0 @@
-require "rails_helper"
-
-RSpec.describe ApprovalsController, type: :controller do
-  render_views
-
-  let(:user) { create(:user) }
-  let(:account) { user.primary_account }
-  let(:project) { create(:project, account: user.primary_account) }
-
-  before do
-    sign_in user
-  end
-
-  describe "#new" do
-    let!(:medical_release) { create(:medical_release, project: project) }
-
-    it "responds successfully" do
-      get :new, params: { medical_release_id: medical_release }
-
-      expect(response).to be_successful
-    end
-  end
-
-  describe "#create" do
-    it "changes approval status successfully" do
-      medical_release = create(:medical_release, project: project)
-
-      expect(MedicalRelease.last.approved?).to eq false
-
-      post :create, params: { medical_release_id: medical_release }
-
-      expect(response).to redirect_to [project, :medical_releases]
-      expect(MedicalRelease.last.approved?).to eq true
-    end
-  end
-end

spec/features/user_creates_task_request_spec.rb

@@ -21,7 +21,6 @@ RSpec.feature 'User creates task request', type: :feature do
     click_on 'Create Task request'
 
     expect(page).to have_content task_created_message
-    expect(page).to have_link("Back")
   end
 
   scenario 'user can view task request details' do
@@ -90,4 +89,4 @@ RSpec.feature 'User creates task request', type: :feature do
   def task_created_message
     t 'task_requests.create.success_message'
   end
-end
+end

spec/features/user_manages_contract_templates_spec.rb

@@ -35,24 +35,6 @@ RSpec.feature 'User manages contract templates', type: :feature do
     expect(page).to have_content(create_contract_template_success_message)
   end
 
-  scenario 'creating new release template, all release types accept signature legal text' do
-    all_release_types = ['Acquired Media', 'Appearance', 'Location', 'Material', 'Medical', 'Misc', 'Talent']
-
-    all_release_types.each do |release_type|
-      visit new_project_contract_template_path(project)
-
-      dropdown_selection = "#{release_type} Release"
-      select dropdown_selection, from: 'Release type'
-
-      fill_in 'Name', with: "My #{release_type} template"
-      fill_in_trix signature_legal_text_field, with: 'LL'
-
-      expect do
-        click_on 'Create Release Template'
-      end.to change(ContractTemplate, :count).by(1)
-    end
-  end
-
   scenario 'medical release template has a guardian clause field' do
     visit new_project_contract_template_path(project)
 
@@ -240,7 +222,7 @@ RSpec.feature 'User manages contract templates', type: :feature do
 
     click_on 'Manage'
     accept_alert do
-      click_on 'Delete'
+      click_on 'Archive'
     end
 
     expect(page).to have_content('The release template has been archived')
@@ -291,7 +273,7 @@ RSpec.feature 'User manages contract templates', type: :feature do
       visit project_contract_templates_path(project)
 
       click_on 'Manage'
-      expect(page).to have_content('Delete')
+      expect(page).to have_content('Archive')
     end
 
     it 'does not show create release button on splash page' do
@@ -328,10 +310,6 @@ RSpec.feature 'User manages contract templates', type: :feature do
     'contract_template_guardian_clause_trix_input_contract_template'
   end
 
-  def signature_legal_text_field
-    'contract_template_signature_legal_text'
-  end
-
   def create_contract_template_success_message
     'The release template has been created'
   end
@@ -351,8 +329,4 @@ RSpec.feature 'User manages contract templates', type: :feature do
   def create_release_template
     t 'contract_templates.splash.actions.create_template'
   end
-
-  def signature_legal_text_trix_field
-    'Signature legal text'
-  end
 end

spec/features/user_managing_acquired_media_releases_spec.rb

@@ -41,13 +41,6 @@ feature "User managing acquired_media releases" do
       expect(AcquiredMediaRelease.last.categories).to include("Still Photograph")
       expect(page).to have_content("Your release was successfully submitted. Thank you.")
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_acquired_media_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context "when signed in" do
@@ -364,8 +357,4 @@ feature "User managing acquired_media releases" do
   def destroy_release_alert
     t "acquired_media_releases.destroy.alert"
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
 end

spec/features/user_managing_appearance_releases_spec.rb

@@ -137,13 +137,6 @@ feature 'User managing appearance releases' do
       expect(page).to have_content(successful_submission_message)
       expect(AppearanceRelease.last.guardian_2_first_name).to eq 'Second'
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_appearance_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context 'when signed in' do
@@ -713,8 +706,4 @@ feature 'User managing appearance releases' do
   def guardian_2_photo_heading
     t 'appearance_releases.form.photos.guardian_2_photo.heading'
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
 end

spec/features/user_managing_location_releases_spec.rb

@@ -71,13 +71,6 @@ feature "User managing location releases" do
       expect(page).to have_content("Your release was successfully submitted. Thank you.")
       expect(LocationRelease.last.photos.attached?).to eq true
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_location_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context "when signed in" do
@@ -382,8 +375,4 @@ feature "User managing location releases" do
     select "Other", from: "Restriction"
     fill_in "Describe other restrictions", with: "Test"
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
 end

spec/features/user_managing_material_releases_spec.rb

@@ -84,13 +84,6 @@ feature "User managing material releases" do
       click_button submit_release_button
       expect(page).to have_content success_submit_message
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_material_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context "when signed in" do
@@ -389,8 +382,4 @@ feature "User managing material releases" do
     click_button submit_release_button
     expect(page).not_to have_content success_submit_message
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
 end

spec/features/user_managing_medical_releases_spec.rb

@@ -168,13 +168,6 @@ feature "User managing medical releases" do
       expect(page).to have_content(successful_submission_message)
 
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_medical_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context "when signed in as account manager" do
@@ -182,18 +175,6 @@ feature "User managing medical releases" do
       sign_in current_user
     end
 
-    scenario "Approved releases have checkmark and non-approved releases don't have checkmarks" do
-      create(:medical_release_with_contract_template, :native, project: project)
-
-      visit project_medical_releases_path(project)
-      expect(page).to have_css('i.fa.fa-check-circle.fa-2x', count: 0)
-
-      create(:medical_release_with_contract_template, :native, project: project, approved_by_user_email: "some@email.com", approved_at: DateTime.now)
-      visit project_medical_releases_path(project)
-
-      expect(page).to have_css('i.fa.fa-check-circle.fa-2x', count: 1)
-    end
-
     scenario "Download All is visible" do
       create(:medical_release_with_contract_template, :native, project: project)
       create(:medical_release_with_contract_template, :non_native, project: project)
@@ -212,15 +193,6 @@ feature "User managing medical releases" do
       expect(page).to have_link("Download", exact: true, count: 2)
     end
 
-    scenario "Review action in Manage menu is visible" do
-      create(:medical_release_with_contract_template, :native, project: project)
-      create(:medical_release_with_contract_template, :non_native, project: project)
-
-      visit project_medical_releases_path(project)
-
-      expect(page).to have_link(review_action, exact: true)
-    end
-
     scenario "Downloading PDF of native medical release is possible" do
       native_release = create(:medical_release_with_contract_template, :native, project: project)
 
@@ -229,64 +201,6 @@ feature "User managing medical releases" do
       click_link *view_release_pdf_link_for(native_release)
       expect(content_type).to eq('application/pdf')
     end
-
-    scenario "Reviewing release" do
-      create(:medical_release_with_contract_template, :native, project: project)
-
-      visit project_medical_releases_path(project)
-
-      click_link review_action
-
-      expect(page).to have_content review_page_heading
-      expect(page).to have_content approve_button
-    end
-
-    scenario 'When viewing the contract PDF of approved release there is page for office use only' do
-      medical_release = create(:medical_release_with_contract_template,
-                               :native,
-                               project: project,
-                               person_first_name: 'Jane',
-                               person_last_name: 'Doe',
-                               approved_by_user_name: "Big Joe",
-                               approved_by_user_email: "some@email.com",
-                               approved_at: DateTime.now)
-
-      sign_in(current_user)
-      visit project_medical_releases_path(project)
-      click_link *view_release_pdf_link_for(medical_release)
-
-      expect(content_type).to eq('application/pdf')
-      expect(content_disposition).to include('inline')
-      expect(pdf_body).to have_content for_office_use_only.upcase
-      expect(pdf_body).to have_content producer_label
-      expect(pdf_body).to have_content production_label
-      expect(pdf_body).to have_content employee_issued_to_label
-      expect(pdf_body).to have_content issued_by_label
-      expect(pdf_body).to have_content date_issued
-      expect(pdf_body).to have_content 'Big Joe'
-    end
-
-    scenario 'When viewing the contract PDF of not approved release there is no page for office use only' do
-      medical_release = create(:medical_release_with_contract_template,
-                               :native,
-                               project: project,
-                               person_first_name: 'Jane',
-                               person_last_name: 'Doe')
-
-      sign_in(current_user)
-      visit project_medical_releases_path(project)
-      click_link *view_release_pdf_link_for(medical_release)
-
-      expect(content_type).to eq('application/pdf')
-      expect(content_disposition).to include('inline')
-      expect(pdf_body).not_to have_content for_office_use_only.upcase
-      expect(pdf_body).not_to have_content producer_label
-      expect(pdf_body).not_to have_content production_label
-      expect(pdf_body).not_to have_content employee_issued_to_label
-      expect(pdf_body).not_to have_content issued_by_label
-      expect(pdf_body).not_to have_content date_issued
-      expect(pdf_body).not_to have_content 'Big Joe'
-    end
   end
 
   context "when the user is manager(project manager)" do
@@ -314,15 +228,6 @@ feature "User managing medical releases" do
       expect(page).to have_link("Download", exact: true, count: 0)
     end
 
-    scenario "Review action in Manage menu is not visible" do
-      create(:medical_release_with_contract_template, :native, project: project)
-      create(:medical_release_with_contract_template, :non_native, project: project)
-
-      visit project_medical_releases_path(project)
-
-      expect(page).not_to have_link(review_action, exact: true)
-    end
-
     scenario "Downloading PDF of native medical release is not possible" do
       native_release = create(:medical_release_with_contract_template, :native, project: project)
 
@@ -367,15 +272,6 @@ feature "User managing medical releases" do
       expect(page).to have_link("Download", exact: true, count: 0)
     end
 
-    scenario "Review action in Manage menu is not visible" do
-      create(:medical_release_with_contract_template, :native, project: project)
-      create(:medical_release_with_contract_template, :non_native, project: project)
-
-      visit project_medical_releases_path(project)
-
-      expect(page).not_to have_link(review_action, exact: true)
-    end
-
     scenario "Downloading PDF of native medical release is not possible" do
       native_release = create(:medical_release_with_contract_template, :native, project: project)
 
@@ -551,44 +447,4 @@ feature "User managing medical releases" do
   def answer_field_for_question(number)
     "medical_release[question_#{number}_answer]"
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
-
-  def review_action
-    t 'medical_releases.medical_release.actions.review'
-  end
-
-  def review_page_heading
-    t 'approvals.new.heading'
-  end
-
-  def approve_button
-    t 'approvals.new.actions.approve'
-  end
-
-  def for_office_use_only
-    t 'contracts.for_office_use_only.heading'
-  end
-
-  def producer_label
-    t 'contracts.for_office_use_only.description_labels.producer'
-  end
-
-  def production_label
-    t 'contracts.for_office_use_only.description_labels.production'
-  end
-
-  def employee_issued_to_label
-    t 'contracts.for_office_use_only.description_labels.employee_issued_to'
-  end
-
-  def issued_by_label
-    t 'contracts.for_office_use_only.description_labels.issued_by'
-  end
-
-  def date_issued
-    t 'contracts.for_office_use_only.description_labels.date_issued'
-  end
 end

spec/features/user_managing_misc_releases_spec.rb

@@ -48,13 +48,6 @@ feature "User managing misc releases" do
         expect(field).to be_visible
       end
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_misc_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context "when signed in as account manager" do
@@ -206,8 +199,4 @@ feature "User managing misc releases" do
   def person_is_minor_checkbox
     'misc_release_minor'
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
 end

spec/features/user_managing_talent_releases_spec.rb

@@ -126,13 +126,6 @@ feature "User managing talent releases" do
       expect(TalentRelease.last.guardian_2_photo.attached?).to eq true
       expect(TalentRelease.last.guardian_2_name).to eq "Second Guardian"
     end
-
-    scenario "creating a release, if contract template contains signature legal language, it is shown" do
-      contract_template = create(:contract_template, project: project, signature_legal_text: dummy_signature_legal_text)
-      visit new_account_project_contract_template_talent_release_path(project.account, project, contract_template)
-
-      expect(page).to have_content dummy_signature_legal_text
-    end
   end
 
   context "when signed in" do
@@ -562,8 +555,4 @@ feature "User managing talent releases" do
     fill_in guardian_address_state_field, with: "NY"
     fill_in guardian_address_zip_field, with: '1000'
   end
-
-  def dummy_signature_legal_text
-    'Some signature legal language'
-  end
 end

spec/jobs/generate_contracts_zip_job_spec.rb

@@ -11,7 +11,7 @@ describe GenerateContractsZipJob do
     dir = Rails.root.join("spec", "fixtures", "files")
     files = ["contract.pdf", "AppearanceRelease.pdf"]
     # Attachments in the test environment do not persist to cloud storage
-    # Therefore we want to stub calls to `open` with a cloud storage URL
+    # Therefore we want to stub calls to `open` with a cloud storage URL 
     allow_any_instance_of(ReleaseContractCollectionService).to receive(:open).and_return(StringIO.new("file data"))
     allow_any_instance_of(ReleaseContractCollectionService).to receive(:build).and_yield(dir, files)
   end
@@ -35,38 +35,6 @@ describe GenerateContractsZipJob do
       expect(download.file).to be_attached
     end
 
-    it "generates ZIP containing CSV file with all releases data for all release types" do
-      release_types = %w[AcquiredMediaRelease AppearanceRelease LocationRelease MaterialRelease MedicalRelease MiscRelease MusicRelease TalentRelease]
-      create_releases_for_all_types
-
-      release_types.each do |type|
-        lowercase_plural = type.constantize.model_name.plural
-        GenerateContractsZipJob.perform_now(project, download, type, project.public_send(lowercase_plural).ids)
-
-        generated_zip = download.file.blob.download
-        csv_file_name = "#{project.name.parameterize}_#{lowercase_plural.gsub('_', '-')}.csv"
-        Zip::InputStream.open(StringIO.new(generated_zip)) do |io|
-          while entry = io.get_next_entry
-            next unless entry.name == csv_file_name
-
-            csv_file = entry.get_input_stream.read
-
-            release_class = Object.const_get type
-            release_headers = release_class.csv_headers
-
-            release_headers.each do |header|
-              expect(csv_file).to match header
-            end
-          end
-
-          dummy_zip_file_name = "#{project.name.parameterize}_#{lowercase_plural.gsub('_', '-')}.zip"
-          if File.exist?(file_fixture(dummy_zip_file_name))
-            File.delete(file_fixture(dummy_zip_file_name))
-          end
-        end
-      end
-    end
-
     context "When there are errors" do
       let(:error) { StandardError.new("Contracts or contract templates not found.") }
 
@@ -74,10 +42,10 @@ describe GenerateContractsZipJob do
         allow(ProjectsChannel).to receive(:broadcast_download_generation_update).with(download, I18n.t("contract_downloads.download.failure"))
         allow_any_instance_of(ReleaseContractCollectionService).to receive(:build).and_raise(StandardError, "Contracts or contract templates not found")
       end
-
+  
       it "updates status to 'failure' and sends user a notification" do
         GenerateContractsZipJob.perform_now(project, download, "AppearanceRelease", project.appearance_releases.ids)
-
+  
         expect(download.status).to eq "failure"
         expect(ProjectsChannel).to have_received(:broadcast_download_generation_update).with(download, I18n.t("contract_downloads.download.failure"))
       end
@@ -88,21 +56,6 @@ describe GenerateContractsZipJob do
     # Delete the file created in fixture.
     # Or the tests will fail on next run due to already existing files in existing zip.
     path = Rails.root.join("spec", "fixtures", "files")
-    if File.exists? "#{path}/my-video-project_appearance-releases.zip"
-      File.delete("#{path}/my-video-project_appearance-releases.zip")
-    end
-  end
-
-  private
-
-  def create_releases_for_all_types
-    create(:acquired_media_release_with_contract_template, :native, project: project)
-    create(:appearance_release_with_contract_template, :native, project: project, person_name: "John Doe")
-    create(:location_release_with_contract_template, :native, project: project)
-    create(:material_release_with_contract_template, :native, project: project)
-    create(:medical_release_with_contract_template, :native, project: project)
-    create(:misc_release_with_contract_template, :native, project: project)
-    create(:music_release_with_contract_template, project: project)
-    create(:talent_release_with_contract_template, :native, project: project)
+    File.delete("#{path}/my-video-project_appearance-releases.zip") if File.exists? "#{path}/my-video-project_appearance-releases.zip"
   end
 end