fix MR comments

app/controllers/contract_templates_controller.rb

@@ -7,6 +7,7 @@ class ContractTemplatesController < ApplicationController
 
   before_action :set_project, except: [:destroy]
   before_action :set_contract_template, only: [:destroy]
+  before_action :show_splash_screen, only: :index
 
   def index
     @contract_templates = contract_templates.non_archived.order_by_name.paginate(page: params[:page])
@@ -38,6 +39,10 @@ class ContractTemplatesController < ApplicationController
 
   private
 
+  def show_splash_screen
+    render :splash if contract_templates.non_archived.count.zero?
+  end
+
   def set_contract_template
     @contract_template = authorize contract_templates.find(params[:id])
   end

app/controllers/videos_controller.rb

@@ -3,7 +3,6 @@ class VideosController < ApplicationController
 
   before_action :set_project, only: [:index, :new, :create, :landing]
   before_action :set_video, only: [:edit, :update]
-  before_action :show_splash_screen, only: :index
 
   def landing
     authorize Video, :new?
@@ -61,10 +60,6 @@ class VideosController < ApplicationController
 
   private
 
-  def show_splash_screen
-    render :splash if videos.count.zero?
-  end
-
   def set_project
     @project = policy_scope(Project).find(params[:project_id])
   end

app/views/contract_templates/splash.html.erb

@@ -1,13 +1,13 @@
 <div class="d-flex flex-row">
   <div class="d-flex flex-column">
-    <%= product_wordmark :deliver_me, prefix: t('.headings.welcome'), class: "h2" %>
+    <%= product_wordmark :release_me, prefix: t('.headings.welcome'), class: "h2" %>
     <p class="text-muted"><%= t '.headings.subtitle' %>
   </div>
 
   <%= 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? %>
+  <% if policy(ContractTemplate).new? %>
-    <%= link_to t(".actions.upload_video"), [:new, @project, :video], class: "btn btn-success border align-self-center h-50 pb-2" %>
+    <%= link_to t(".actions.create_template"), [:new, @project, :contract_template], class: "btn btn-success border align-self-center h-50 pb-2" %>
-  <% end %>
+<% end %>
 </div>
 
 <hr>
@@ -36,10 +36,10 @@
         <div class="col">
           <h3><%= t '.headings.how_it_works' %></h3>
           <ol>
-            <li><%= t '.list_items.import_video' %></li>
+            <li><%= t '.list_items.create_releases' %></li>
-            <li><%= t '.list_items.import_EDLs' %></li>
+            <li><%= t '.list_items.download_mobile_app' %></li>
-            <li><%= t '.list_items.AI_generates_documents' %></li>
+            <li><%= t '.list_items.print_QR_code' %></li>
-            <li><%= t '.list_items.download_documents' %></li>
+            <li><%= t '.list_items.releases_automatically_organized' %></li>
           </ol>
         </div>
       </div>
@@ -47,10 +47,9 @@
         <div class="col">
           <h3><%= t '.headings.benefits' %></h3>
           <ul class="fa-ul ml-5">
-            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.reduces_labor_cost'))) %>
+            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.all_releases_available'))) %>
-            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.more_accurate'))) %>
+            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.manage_large_audience'))) %>
-            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.simplifies_cue_sheets'))) %>
+            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.add_tags_and_notes'))) %>
-            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.production_elements_logs'))) %>
           </ul>
         </div>
       </div>

config/locales/en.yml

@@ -252,6 +252,23 @@ en:
         signed_release_count: No. Signed Releases
     new:
       heading: New Release Template
+    splash:
+      actions:
+        book_demo: Schedule a Demo
+        create_template: Create New Release Template
+      headings:
+        benefits: Benefits
+        how_it_works: How It Works
+        subtitle: The easiest way to create and manage releases.
+        welcome: Welcome to
+      list_items:
+        add_tags_and_notes: Add tags and notes to releases for added organization and searchability
+        all_releases_available: All release types available including Covid Medical, Appearance, Location and more
+        create_releases: Create releases for your video shoot
+        download_mobile_app: Download the ME Suite mobile app
+        manage_large_audience: Easily manage large audiences with the crowd control feature
+        print_QR_code: Print out release QR codes
+        releases_automatically_organized: Releases are automatically organized as they’re submitted
   contracts:
     medical_questionnaire:
       heading: Medical Questionnaire
@@ -1273,24 +1290,6 @@ en:
     new:
       heading: Upload Video
       subheading: 2 of 2 Files
-    splash:
-      actions:
-        book_demo: Schedule a Demo
-        upload_video: Upload New Video
-      headings:
-        benefits: Benefits
-        how_it_works: How It Works
-        subtitle: Automate your deliverable documents
-        welcome: Welcome to
-      list_items:
-        AI_generates_documents: AI generates deliverable documents
-        download_documents: Download deliverable documents
-        import_EDLs: Import your EDLs
-        import_video: Import your video
-        more_accurate: More accurate than human-generated reports
-        production_elements_logs: Production Elements Logs, and more
-        reduces_labor_cost: Reduces labor costs
-        simplifies_cue_sheets: Simplifies Music Cue Sheets, Graphic Cue Sheets
     update:
       notice: The video has been updated
     video:

config/locales/es.yml

@@ -70,6 +70,23 @@ es:
         heading: Legal (ES)
       release_info:
         heading: Release Info (ES)
+    splash:
+      actions:
+        book_demo: Schedule a Demo (ES)
+        create_template: Create New Release Template (ES)
+      headings:
+        benefits: ""
+        how_it_works: How It Works (ES)
+        subtitle: The easiest way to create and manage releases. (ES)
+        welcome: Welcome to (ES)
+      list_items:
+        add_tags_and_notes: Add tags and notes to releases for added organization and searchability (ES)
+        all_releases_available: All release types available including Covid Medical, Appearance, Location and more (ES)
+        create_releases: Create releases for your video shoot (ES)
+        download_mobile_app: Download the ME Suite mobile app (ES)
+        manage_large_audience: Easily manage large audiences with the crowd control feature (ES)
+        print_QR_code: Print out release QR codes (ES)
+        releases_automatically_organized: Releases are automatically organized as they’re submitted (ES)
   contracts:
     photos:
       guardian_2_photo_heading: Second guardian photo (ES)
@@ -363,22 +380,3 @@ es:
         manage: Manage (ES)
         open_deliverable: Open Deliverable (ES)
         read_more: read more (ES)
-  videos:
-    splash:
-      actions:
-        book_demo: Schedule a Demo (ES)
-        upload_video: Upload New Video (ES)
-      headings:
-        benefits: Benefits (ES)
-        how_it_works: How It Works (ES)
-        subtitle: Automate your deliverable documents (ES)
-        welcome: Welcome to (ES)
-      list_items:
-        AI_generates_documents: AI generates deliverable documents (ES)
-        download_documents: Download deliverable documents (ES)
-        import_EDLs: Import your EDLs (ES)
-        import_video: Import your video (ES)
-        more_accurate: More accurate than human-generated reports (ES)
-        production_elements_logs: Production Elements Logs, and more (ES)
-        reduces_labor_cost: Reduces labor costs (ES)
-        simplifies_cue_sheets: Simplifies Music Cue Sheets, Graphic Cue Sheets (ES)

db/structure.sql

@@ -9,6 +9,20 @@ SET xmloption = content;
 SET client_min_messages = warning;
 SET row_security = off;
 
+--
+-- Name: plpgsql; Type: EXTENSION; Schema: -; Owner: -
+--
+
+CREATE EXTENSION IF NOT EXISTS plpgsql WITH SCHEMA pg_catalog;
+
+
+--
+-- Name: EXTENSION plpgsql; Type: COMMENT; Schema: -; Owner: -
+--
+
+COMMENT ON EXTENSION plpgsql IS 'PL/pgSQL procedural language';
+
+
 --
 -- Name: fuzzystrmatch; Type: EXTENSION; Schema: -; Owner: -
 --
@@ -317,8 +331,8 @@ CREATE TABLE public.appearance_releases (
     person_last_name character varying,
     guardian_first_name character varying,
     guardian_last_name character varying,
-    identifier character varying,
     guardian_email character varying,
+    identifier character varying,
     person_address_street2 character varying,
     person_address_city character varying,
     person_address_state character varying,
@@ -652,6 +666,15 @@ CREATE SEQUENCE public.contract_templates_id_seq
 ALTER SEQUENCE public.contract_templates_id_seq OWNED BY public.contract_templates.id;
 
 
+--
+-- Name: data_migrations; Type: TABLE; Schema: public; Owner: -
+--
+
+CREATE TABLE public.data_migrations (
+    version character varying NOT NULL
+);
+
+
 --
 -- Name: directories; Type: TABLE; Schema: public; Owner: -
 --
@@ -1378,7 +1401,6 @@ CREATE TABLE public.settings (
 --
 
 CREATE SEQUENCE public.settings_id_seq
-    AS integer
     START WITH 1
     INCREMENT BY 1
     NO MINVALUE
@@ -1414,7 +1436,6 @@ CREATE TABLE public.taggings (
 --
 
 CREATE SEQUENCE public.taggings_id_seq
-    AS integer
     START WITH 1
     INCREMENT BY 1
     NO MINVALUE
@@ -1445,7 +1466,6 @@ CREATE TABLE public.tags (
 --
 
 CREATE SEQUENCE public.tags_id_seq
-    AS integer
     START WITH 1
     INCREMENT BY 1
     NO MINVALUE
@@ -1806,9 +1826,9 @@ CREATE TABLE public.zoom_meetings (
     api_meeting_id character varying,
     created_at timestamp(6) without time zone NOT NULL,
     updated_at timestamp(6) without time zone NOT NULL,
+    status integer DEFAULT 0,
     zoom_user_id bigint,
-    project_id bigint,
+    project_id bigint
-    status integer DEFAULT 0
 );
 
 
@@ -2285,6 +2305,14 @@ ALTER TABLE ONLY public.contract_templates
     ADD CONSTRAINT contract_templates_pkey PRIMARY KEY (id);
 
 
+--
+-- Name: data_migrations data_migrations_pkey; Type: CONSTRAINT; Schema: public; Owner: -
+--
+
+ALTER TABLE ONLY public.data_migrations
+    ADD CONSTRAINT data_migrations_pkey PRIMARY KEY (version);
+
+
 --
 -- Name: directories directories_pkey; Type: CONSTRAINT; Schema: public; Owner: -
 --

spec/controllers/contract_templates_controller_spec.rb

@@ -19,7 +19,7 @@ describe ContractTemplatesController do
       expect(response).to be_successful
     end
 
-    it 'renders content' do
+    it 'renders content if there are contract templates' do
       contract_template = create(:contract_template,
                                  name: 'My Contract Template', fee: 50, release_type: 'appearance',
                                  project: project)
@@ -39,10 +39,11 @@ describe ContractTemplatesController do
     end
 
     context 'when there are no contract templates' do
-      it 'renders an empty message' do
+      it 'renders splash screen' do
         get :index, params: { project_id: project }
 
-        expect(response.body).to have_content('Release Templates will appear here')
+        expect(response.body).to have_link "Create New Release Template"
+        expect(response.body).to have_link schedule_demo
       end
     end
 
@@ -50,6 +51,7 @@ describe ContractTemplatesController do
       let(:current_user) { create(:user, :associate) }
 
       it 'does not show the new contract template button' do
+        create(:contract_template, project: project)
         get :index, params: { project_id: project }
 
         expect(response.body).not_to have_link('Create New Release Template')
@@ -166,4 +168,8 @@ describe ContractTemplatesController do
       restriction_text: 'restrictions'
     }
   end
+
+  def schedule_demo
+    t 'contract_templates.splash.actions.book_demo'
+  end
 end

spec/controllers/videos_controller_spec.rb

@@ -20,7 +20,6 @@ RSpec.describe VideosController, type: :controller do
     end
 
     it "has a search form" do
-      create(:video, project: project)
       get :index, params: { project_id: project }
 
       expect(response.body).to have_button("search-button")
@@ -44,7 +43,7 @@ RSpec.describe VideosController, type: :controller do
     end
 
 
-    it "renders content if there are existing videos" do
+    it "renders content" do
       video = create(:video, project: project, name: "My Video", number: "001", created_at: 1.day.ago)
 
       get :index, params: { project_id: project }
@@ -59,11 +58,10 @@ RSpec.describe VideosController, type: :controller do
     end
 
     context "when there are no records" do
-      it "renders splash screen" do
+      it "renders an empty message" do
         get :index, params: { project_id: project }
 
-        expect(response.body).to have_link "Upload New Video"
+        expect(response.body).to have_content("Videos will appear here")
-        expect(response.body).to have_link schedule_demo
       end
     end
 
@@ -279,8 +277,4 @@ RSpec.describe VideosController, type: :controller do
   def video_update_params
     attributes_for(:video, :with_graphics_only_edl_file, :with_audio_only_edl_file, name: "Test Video").except(:file)
   end
-
-  def schedule_demo
-    t 'videos.splash.actions.book_demo'
-  end
 end

spec/features/user_imports_release_templates_spec.rb

@@ -13,6 +13,7 @@ feature "User imports release templates", type: :feature do
   end
 
   scenario "importing two existing templates into a project" do
+    create(:contract_template, project: project)
     visit project_contract_templates_path(project)
     click_on "Import Release Template"
     select_templates([project_one_template.id, project_two_template.id])
@@ -31,6 +32,7 @@ feature "User imports release templates", type: :feature do
   end
 
   scenario "searching for a template", js: true do
+    create(:contract_template, project: project)
     visit project_contract_templates_path(project)
     click_on "Import Release Template"
     fill_in "query", with: "Second"

spec/features/user_manages_contract_templates_spec.rb

@@ -11,6 +11,13 @@ RSpec.feature 'User manages contract templates', type: :feature do
     sign_in(current_user)
   end
 
+  scenario 'splash page is shown if tehre are no contract templates' do
+    visit project_contract_templates_path(project)
+
+    expect(page).to have_content schedule_demo
+    expect(page).to have_content create_release_template
+  end
+
   scenario 'creating a new release template' do
     visit new_project_contract_template_path(project)
 
@@ -223,12 +230,20 @@ RSpec.feature 'User manages contract templates', type: :feature do
     let(:current_user) { create(:user, :associate) }
 
     it 'does not show management buttons for release templates' do
+      create(:contract_template, project: project)
       visit project_contract_templates_path(project)
 
       expect(page).not_to have_content('Create New Release Template')
       expect(page).not_to have_content(import_template_button)
       expect(page).not_to have_content('Delete')
     end
+
+    it 'does not show create release button on splash page' do
+      visit project_contract_templates_path(project)
+
+      expect(page).to have_content schedule_demo
+      expect(page).not_to have_content create_release_template
+    end
   end
 
   context 'When the user is account manager' do
@@ -242,6 +257,13 @@ RSpec.feature 'User manages contract templates', type: :feature do
       click_on 'Manage'
       expect(page).to have_content('Archive')
     end
+
+    it 'does not show create release button on splash page' do
+      visit project_contract_templates_path(project)
+
+      expect(page).to have_content schedule_demo
+      expect(page).to have_content create_release_template
+    end
   end
 
   private
@@ -273,4 +295,12 @@ RSpec.feature 'User manages contract templates', type: :feature do
   def fill_hidden(id, with:)
     find(:xpath, "//input[@id='#{id}']", visible: false).set with
   end
+
+  def schedule_demo
+    t 'contract_templates.splash.actions.book_demo'
+  end
+
+  def create_release_template
+    t 'contract_templates.splash.actions.create_template'
+  end
 end

spec/features/user_managing_videos_spec.rb

@@ -1,8 +1,8 @@
 require "rails_helper"
 
 feature "User managing videos" do
-  let(:current_user) { create(:user, :manager) }
+  let(:current_user) { create(:user) }
-  let(:project) { create(:project, members: current_user, account: current_user.primary_account) }
+  let(:project) { create(:project, account: current_user.primary_account) }
   let!(:video) do
     create(:video,
            :with_graphics_only_edl_file,
@@ -17,14 +17,6 @@ feature "User managing videos" do
     sign_in current_user
   end
 
-  scenario "splash page is shown if there are no existing videos" do
-    Video.delete_all
-    visit project_videos_path(project)
-
-    expect(page).to have_content schedule_demo
-    expect(page).to have_content upload_new_video
-  end
-
   scenario "creating a video", js: true do
     visit project_videos_path(project)
     click_link "Upload New Video"
@@ -104,30 +96,6 @@ feature "User managing videos" do
     expect(page).to have_content("Second Video")
   end
 
-  context 'When the user is associate' do
-    let(:current_user) { create(:user, :associate) }
-
-    it 'does show button to upload new video' do
-      Video.delete_all
-      visit project_videos_path(project)
-
-      expect(page).to have_content schedule_demo
-      expect(page).to have_content upload_new_video
-    end
-  end
-
-  context 'When the user is account manager' do
-    let(:current_user) { create(:user, :account_manager) }
-
-    it 'does show button to upload new video' do
-      Video.delete_all
-      visit project_videos_path(project)
-
-      expect(page).to have_content schedule_demo
-      expect(page).to have_content upload_new_video
-    end
-  end
-
   private
 
   def fill_in_video_fields(data)
@@ -147,12 +115,4 @@ feature "User managing videos" do
   def update_video_notice
     t "videos.update.notice"
   end
-
-  def schedule_demo
-    t 'videos.splash.actions.book_demo'
-  end
-
-  def upload_new_video
-    t 'videos.splash.actions.upload_video'
-  end
 end