update video sources

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/contract_templates_controller.rb

@@ -62,8 +62,7 @@ class ContractTemplatesController < ApplicationController
   def contract_template_params
     params
       .require(:contract_template)
-      .permit(:name, :release_type, :body, :guardian_clause,
+      .permit(:name, :release_type, :body, :guardian_clause, :fee,
-              :signature_legal_text, :fee,
               :applicable_medium_id, :applicable_medium_text,
               :territory_id, :territory_text,
               :term_id, :term_text,

app/jobs/generate_contracts_zip_job.rb

@@ -22,7 +22,7 @@ class GenerateContractsZipJob < ApplicationJob
         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)

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/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/views/contract_templates/_contract_template.html.erb

@@ -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/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

@@ -97,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("check", text: "Done"), [@project, :task_requests], class: "btn btn-primary" %>

config/locales/en.yml

@@ -364,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:

config/locales/es.yml

@@ -160,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)

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,8 +136,8 @@ Rails.application.routes.draw do
       end
     end
 
-    RELEASES = [:acquired_media_releases, :appearance_releases, :talent_releases, :material_releases, :medical_releases, :misc_releases, :location_releases].freeze
+    RELEASES = [:acquired_media_releases, :appearance_releases, :talent_releases, :material_releases, :location_releases]
-    ALL_RELEASES = RELEASES + [:music_releases]
+    ALL_RELEASES = RELEASES + [:music_releases, :medical_releases, :misc_releases]
 
     ALL_RELEASES.each do |release|
       resources release, only: [], concerns: :taggable
@@ -158,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

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/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("Done")
   end
 
   scenario 'user can view task request details' do

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
@@ -454,8 +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
 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