add specs to check if United States is default selected country on public signing form

app/assets/javascripts/channels/broadcasts.coffee

@@ -1,13 +1,8 @@
 $(document).on "turbolinks:load", ->
-  # Only connect if a broadcast-token meta tag is present
+  subscribeToBroadcast = (broadcastToken) -> App.cable.subscriptions.create { channel: "BroadcastsChannel", token: broadcastToken },
-  broadcast_meta = document.querySelector("meta[name=broadcast-token]")
-  return unless broadcast_meta
-
-  broadcastToken = broadcast_meta.getAttribute("content")
-
-  App.public = App.cable.subscriptions.create { channel: "BroadcastsChannel", token: broadcastToken },
     connected: ->
       # Called when the subscription is ready for use on the server
+      console.info "Subscribed to channel for broadcast:#{broadcastToken}"
 
     disconnected: ->
       # Called when the subscription has been terminated by the server
@@ -17,6 +12,7 @@ $(document).on "turbolinks:load", ->
       switch data.event
         when "broadcast_stream_update" then @refreshBroadcastVideo(data)
         when "stream_recording_ready" then @showBroadcastRecordings(data)
+        when "file_upload_update" then @refreshBroadcastFilesTab(data)
 
     refreshBroadcastVideo: (data) ->
       $("#broadcast_updates").html data.status_content
@@ -37,3 +33,12 @@ $(document).on "turbolinks:load", ->
       $(".flash-message").html data.flash_content
       $("#broadcast_recordings").html data.recordings_content
       $("#broadcast_recordings_nav").html data.recordings_nav_content
+
+    refreshBroadcastFilesTab: (data) ->
+      $("#broadcast_file_list_#{data.broadcast_token}").html data.files_content
+      $("#broadcast_files_pagination_#{data.broadcast_token}").html data.pagination_content
+
+  # Create a channel subscription for every broadcast included in the meta tags
+  get_token = (meta) -> meta.getAttribute("content")
+  broadcast_tokens = (get_token broadcast_meta for broadcast_meta in document.querySelectorAll("meta[name=broadcast-token]"))
+  subscribeToBroadcast token for token in broadcast_tokens

app/assets/javascripts/channels/projects.coffee

@@ -16,6 +16,7 @@ $(document).on "turbolinks:load", ->
       switch data.event
         when "video_status_update" then @showVideoStatusUpdate(data.content)
         when "download_status_update" then @showDownloadStatusUpdate(data.content)
+        when "conference_recording_ready" then @showDownloadStatusUpdate(data.content)
 
     showVideoStatusUpdate: (content) ->
       $("[data-ujs-target='video-analysis-msg']").replaceWith content

app/assets/stylesheets/application.scss

@@ -6,6 +6,11 @@
 @import "dropzone/dist/dropzone";
 @import 'bootstrap-datepicker';
 
+// Global styles
+label {
+  text-transform: capitalize;
+}
+
 // Wordmarks
 .suite-wordmark {
   font-weight: normal;

app/channels/broadcasts_channel.rb

@@ -34,4 +34,15 @@ class BroadcastsChannel < ApplicationCable::Channel
       recordings_content: recordings_content,
       recordings_nav_content: recordings_nav_content
   end
+
+  def self.broadcast_file_upload_updates(broadcast, files, pagination_content)
+    files_content = ApplicationController.render partial: "broadcasts/file", collection: files
+
+    broadcast_to broadcast, {
+      event: :file_upload_update,
+      broadcast_token: broadcast.token,
+      files_content: files_content,
+      pagination_content: pagination_content
+    }
+  end
 end

app/channels/projects_channel.rb

@@ -25,4 +25,12 @@ class ProjectsChannel < ApplicationCable::Channel
     content = ApplicationController.render partial: "application/flash", locals: { flash: flash }
     broadcast_to download.project, event: :download_status_update, content: content
   end
+
+  def self.conference_recording_ready(project, recording)
+    link = ApplicationController.helpers.link_to('Download here.', recording.service_url, target: '_blank', class: 'alert-link')
+    notification = "A recording of your video conference is now available. #{link}"
+    flash = OpenStruct.new(notice: notification)
+    content = ApplicationController.render partial: 'application/flash', locals: { flash: flash }
+    broadcast_to project, event: :conference_recording_ready, content: content
+  end
 end

app/controllers/acquired_media_releases_controller.rb

@@ -61,7 +61,8 @@ class AcquiredMediaReleasesController < ApplicationController
       :name,
       :territory,
       :term,
-      :person_name,
+      :person_first_name,
+      :person_last_name,
       :person_phone,
       :person_email,
       :person_company,

app/controllers/admin/task_requests_controller.rb

@@ -1,32 +0,0 @@
-class Admin::TaskRequestsController < Admin::ApplicationController
-  before_action :set_task_request, only: [:edit, :update]
-
-  def index
-    @task_requests = task_requests.order_by_recent.paginate(page: params[:page])
-  end
-
-  def edit
-  end
-
-  def update
-    if @task_request.update(task_request_params)
-      redirect_to [:admin, :task_requests], notice: t(".notice")
-    else
-      render :edit
-    end
-  end
-
-  private
-
-  def task_request_params
-    params.require(:task_request).permit(:status, :deliverable_url)
-  end
-
-  def task_requests
-    policy_scope TaskRequest
-  end
-
-  def set_task_request
-    @task_request = authorize policy_scope(TaskRequest).find(params[:id])
-  end
-end

app/controllers/broadcasts_controller.rb

@@ -24,18 +24,21 @@ class BroadcastsController < ApplicationController
     end
   end
 
-  def update
-    @broadcast.update(broadcast_params)
-    @files = @broadcast.files.order("created_at DESC").paginate(page: 1)
-  end
-
   def show
     @conference_url = url_for [@broadcast.project, @broadcast, :zoom_meeting]
     @recordings = @broadcast.broadcast_recordings.order_by_recent.paginate(page: params[:page])
-    @files = @broadcast.files.order("created_at DESC").paginate(page: params[:page])
+    @files = @broadcast.files.order("created_at DESC").paginate(page: params[:files_page])
     render layout: 'application'
   end
 
+  def update
+    @broadcast.update(broadcast_params)
+    @files = @broadcast.files.order("created_at DESC").paginate(page: 1)
+
+    pagination_content = ApplicationController.render html: helpers.will_paginate(@files, params: { active_tab: params[:active_tab], page: params[:page], active_files_tab: params[:active_files_tab] })
+    BroadcastsChannel.broadcast_file_upload_updates(@broadcast, @files, pagination_content)
+  end
+
   def destroy
     if @broadcast.destroy
       redirect_to [@project, :broadcasts], alert: t(".alert")
@@ -68,7 +71,10 @@ class BroadcastsController < ApplicationController
 
   def set_multi_view_broadcasts
     authorized_broadcasts = authorize policy_scope(Broadcast).where(id: params[:multi_view_ids]).order_by_recent
-    @multi_view_broadcasts = authorized_broadcasts.map { |b| MultiViewBroadcast.new(b, params[:multi_view_ids]) }
+    @multi_view_broadcasts = authorized_broadcasts.map do |b|
+      files_page = params[:files_page] if params[:active_files_tab] == b.token
+      MultiViewBroadcast.new(b, params[:multi_view_ids], files_page)
+    end
   end
 
   def filtered_broadcasts
@@ -90,17 +96,26 @@ class BroadcastsController < ApplicationController
 
     delegate_missing_to :@broadcast
 
-    def initialize(broadcast, multi_view_ids)
+    def initialize(broadcast, multi_view_ids, paginate_page)
       @broadcast = broadcast
       @multi_view_ids = multi_view_ids
+      @paginate_page = paginate_page
     end
 
     def url
       project_broadcast_path(@broadcast.project, @broadcast, multi_view_ids: @multi_view_ids, locale: I18n.locale)
     end
 
+    def files
+      @broadcast.files.order("created_at DESC").paginate(page: @paginate_page)
+    end
+
     def uid
       id
     end
+
+    def self.model_name
+      Broadcast.model_name
+    end
   end
 end

app/controllers/location_releases_controller.rb

@@ -68,7 +68,8 @@ class LocationReleasesController < ApplicationController
       :territory_id, :territory_text,
       :term_id, :term_text,
       :restriction_id, :restriction_text,
-      :filming_started_on, :filming_ended_on
+      :filming_started_on, :filming_ended_on,
+      :filming_hours
     )
   end
 

app/controllers/password_resets_controller.rb

@@ -18,6 +18,7 @@ class PasswordResetsController < ApplicationController
   end
 
   def edit
+    redirect_to new_session_path, notice: t(".notice") if @user.nil?
   end
 
   def update

app/controllers/public/acquired_media_releases_controller.rb

@@ -45,6 +45,9 @@ class Public::AcquiredMediaReleasesController < Public::BaseController
     params.require(:acquired_media_release).permit(
       :name, 
       :description,
+      :person_first_name,
+      :person_last_name,
+      :person_email,
       :person_title,
       :person_phone,
       :person_fax,

app/controllers/public/broadcasts_controller.rb

@@ -6,7 +6,7 @@ class Public::BroadcastsController < Public::BaseController
     @conference_url = broadcast_zoom_meeting_url(@broadcast.token)
     @multi_view_broadcasts = multi_view_broadcasts
     @recordings = @broadcast.broadcast_recordings.order_by_recent.paginate(page: params[:page])
-    @files = @broadcast.files.order("created_at DESC").paginate(page: params[:page])
+    @files = @broadcast.files.order("created_at DESC").paginate(page: params[:files_page])
 
     render 'broadcasts/show'
   end
@@ -14,6 +14,9 @@ class Public::BroadcastsController < Public::BaseController
   def update
     @broadcast.update(broadcast_params)
     @files = @broadcast.files.order("created_at DESC").paginate(page: 1)
+
+    pagination_content = ApplicationController.render html: helpers.will_paginate(@files,params: { active_tab: params[:active_tab], page: params[:page], active_files_tab: params[:active_files_tab] })
+    BroadcastsChannel.broadcast_file_upload_updates(@broadcast, @files, pagination_content)
   end
 
   private
@@ -26,7 +29,10 @@ class Public::BroadcastsController < Public::BaseController
     Broadcast.
       where(token: params[:multi_view_tokens]).
       order_by_recent.
-      map { |b| MultiViewBroadcast.new(b, params[:multi_view_tokens]) }
+      map do |b|
+        files_page = params[:files_page] if params[:active_files_tab] == b.token
+        MultiViewBroadcast.new(b, params[:multi_view_tokens], files_page)
+      end
   end
 
   def set_broadcast
@@ -38,17 +44,26 @@ class Public::BroadcastsController < Public::BaseController
 
     delegate_missing_to :@broadcast
 
-    def initialize(broadcast, multi_view_tokens)
+    def initialize(broadcast, multi_view_tokens, paginate_page)
       @broadcast = broadcast
       @multi_view_tokens = multi_view_tokens
+      @paginate_page = paginate_page
     end
 
     def url
       broadcast_url(uid, multi_view_tokens: @multi_view_tokens, host: AppHost.new.domain_with_port, locale: I18n.locale)
     end
 
+    def files
+      @broadcast.files.order("created_at DESC").paginate(page: @paginate_page)
+    end
+
     def uid
       token
     end
+
+    def self.model_name
+      Broadcast.model_name
+    end
   end
 end

app/controllers/public/location_releases_controller.rb

@@ -63,7 +63,8 @@ class Public::LocationReleasesController < Public::BaseController
       :person_address_zip, 
       :person_address_country,
       :signature_base64,
-      :locale, :contract_template, :filming_started_on, :filming_ended_on
+      :locale, :contract_template, :filming_started_on, :filming_ended_on,
+      :filming_hours
     )
   end
 

app/controllers/task_requests_controller.rb

@@ -1,74 +0,0 @@
-class TaskRequestsController < ApplicationController
-  layout "project"
-
-  before_action :set_project
-  before_action :build_task_request, only: [:new, :create]
-  before_action :set_task_request, only: [:show, :edit, :update, :cancel]
-
-  def index
-    if params[:completed_only]
-      @task_requests = task_requests.completed.order_by_recent.paginate(page: params[:page])
-    else
-      @task_requests = task_requests.order_by_recent.paginate(page: params[:page])
-    end
-  end
-
-  def new
-  end
-
-  def create
-    @task_request.attributes = task_request_params
-
-    if @task_request.save
-      log_create_analytics
-      redirect_to [@project, :task_requests], notice: t(".notice")
-    else
-      render :new
-    end
-  end
-
-  def show
-  end
-  
-  def edit
-  end
-
-  def update
-    if @task_request.update(task_request_params)
-      redirect_to [@project, :task_requests], notice: t(".notice")
-    else
-      render :edit
-    end
-  end
-
-  def cancel
-    @task_request.cancelled!
-    redirect_to [@project, :task_requests], notice: t(".notice")
-  end
-
-  private
-
-  def task_request_params
-    params.require(:task_request).permit(:description, :deadline, :time_allowed, :additional_notes, files: [])
-  end
-
-  def set_project
-    @project = policy_scope(Project).find(params[:project_id])
-  end
-
-  def set_task_request
-    @task_request = authorize policy_scope(TaskRequest).find(params[:id])
-  end
-
-  def task_requests
-    authorize policy_scope(@project.task_requests)
-  end
-
-  def build_task_request
-    @task_request = authorize @project.task_requests.build
-  end
-
-  def log_create_analytics
-    TrackAnalyticsJob.perform_later(Current.user, Current.account, :track_create_task_request, user_agent: request.user_agent, user_ip: request.remote_ip)
-  end
-end

app/controllers/zoom_notifications_controller.rb

@@ -13,8 +13,11 @@ class ZoomNotificationsController < ApplicationController
       @zoom_meeting.started!
     when 'meeting.ended'
       @zoom_meeting.ended!
+    when 'recording.completed'
+      recording = notification.dig(:payload, :object, :recording_files).first
+      AttachRecordingToZoomMeetingJob.perform_later(@zoom_meeting, recording, notification['download_token'])
     else
-      Rails.logger.info notification_type
+      Rails.logger.info notification_event
       Rails.logger.info notification
     end
 
@@ -23,16 +26,20 @@ class ZoomNotificationsController < ApplicationController
 
   private
 
+  def notification
+    params.to_unsafe_h
+  end
+
   def notification_event
-    params.dig(:event)
+    notification.dig(:event)
   end
 
   def notification_meeting_id
-    params.dig(:payload, :object, :id)
+    notification.dig(:payload, :object, :id)
   end
 
   def notification_host_id
-    params.dig(:payload, :object, :host_id)
+    notification.dig(:payload, :object, :host_id)
   end
 
   def set_zoom_meeting

app/helpers/contact_info_helper.rb

@@ -41,7 +41,7 @@ module ContactInfoHelper
   end
 
   def email_abbr(email)
-    content_tag(:abbr, "E: ", title: "Email") + email
+    content_tag(:abbr, "E: ", title: "Email") + mail_to(email)
   end
 
   def phone_abbr(phone)

app/jobs/attach_recording_to_zoom_meeting_job.rb

@@ -0,0 +1,26 @@
+require 'zoom_gateway'
+class AttachRecordingToZoomMeetingJob < ApplicationJob
+  queue_as :default
+
+  def perform(zoom_meeting, recording_hash, download_token)
+    download_url = "#{recording_hash['download_url']}?access_token=#{download_token}"
+    file = URI.open(download_url)
+    if zoom_meeting.recording.attach(io: file, filename: file_name(zoom_meeting, recording_hash), content_type: 'video/mp4')
+      # Temorarily disabling notifications
+      # if zoom_meeting.project.present?
+      #   ProjectsChannel.conference_recording_ready(zoom_meeting.project, zoom_meeting.recording)
+      # end
+
+      gateway = ZoomGateway.new
+      gateway.delete_recording(zoom_meeting.api_meeting_id, recording_hash['id'])
+    end
+  end
+
+  private
+
+  def file_name(zoom_meeting, recording_hash)
+    start = recording_hash['recording_start'].to_datetime
+    prefix = zoom_meeting.project.present? ? "#{zoom_meeting.project.name}_" : ''
+    "#{prefix}video_conference_date#{start.strftime("%Y-%m-%d")}_Time_#{start.strftime("%T")}.mp4"
+  end
+end

app/models/account.rb

@@ -54,7 +54,7 @@ class Account < ApplicationRecord
         Download.where(project: projects),
         User.joins(:project_memberships).where(project_memberships: { project: projects }),
         Broadcast.where(project: projects),
-        TaskRequest.where(project: projects),
+        ZoomMeeting.where(project: projects),
         self
       ])).sum(:byte_size).to_f
   end
@@ -79,10 +79,6 @@ class Account < ApplicationRecord
     plan_uid.to_s == "me_suite" || plan_uid.to_s == "releaseme"
   end
 
-  def assistme_enabled?
-    plan_uid.to_s == "me_suite" || plan_uid.to_s == "assistme"
-  end
-
   def plan_name
     case plan_uid.to_s
     when "deliverme"
@@ -91,8 +87,6 @@ class Account < ApplicationRecord
       "DirectME"
     when "releaseme"
       "ReleaseME"
-    when "assistme"
-      "AssistME"
     when "me_suite"
       "ME Suite"
     end

app/models/acquired_media_release.rb

@@ -43,7 +43,8 @@ class AcquiredMediaRelease < ApplicationRecord
     person_address_street1 person_address_street2 person_address_city person_address_state person_address_zip person_address_country
   ]
 
-  CATEGORIES = ["Artwork", "Film Footage", "Video Footage", "Still Photograph"].freeze
+  # CATEGORIES = ["Artwork", "Film Footage", "Video Footage", "Still Photograph"].freeze
+  CATEGORIES = ["Film Footage", "Video Footage", "Still Photograph"].freeze
 
   def minor?
     false

app/models/broadcast.rb

@@ -7,8 +7,6 @@ class Broadcast < ApplicationRecord
 
   has_secure_token
 
-  scope :order_by_recent, -> { order(created_at: :desc) }
-
   validates :name, presence: true
 
   enum status: [:created, :active, :idle]

app/models/broadcast_recording.rb

@@ -1,8 +1,6 @@
 class BroadcastRecording < ApplicationRecord
   belongs_to :broadcast
 
-  scope :order_by_recent, -> { order(created_at: :desc) }
-
   delegate :name, to: :broadcast, prefix: :broadcast
 
   validates :asset_uid, uniqueness: true

app/models/concerns/releasable.rb

@@ -6,8 +6,6 @@ module Releasable
   included do
     belongs_to :project, touch: true
     belongs_to :contract_template, optional: true
-
-    scope :order_by_recent, -> { order(created_at: :desc) }
   end
 
   def release_number

app/models/headshot_collection.rb

@@ -5,7 +5,7 @@ class HeadshotCollection
   def self.for_project(project)
     appearance_releases_with_photo = project.appearance_releases.with_person_photo
 
-    new(project.id, appearance_releases_with_photo + project.talent_releases)
+    new(project.headshot_collection_uid, appearance_releases_with_photo + project.talent_releases)
   end
 
   def initialize(collection_uid, releasables)
@@ -23,7 +23,7 @@ class HeadshotCollection
       collection_uid: collection_uid.to_s,
       bucket_name: aws_bucket_name,
       ids_to_images: map_ids_to_images,
-    }
+    }.reject { |_, v| v.blank? }
   end
 
   private

app/models/note.rb

@@ -4,6 +4,4 @@ class Note < ApplicationRecord
   belongs_to :notable, polymorphic: true
 
   validates :content, presence: true
-
-  scope :order_by_recent, -> { order(created_at: :desc) }
 end

app/models/project.rb

@@ -22,7 +22,6 @@ class Project < ApplicationRecord
   has_many :downloads, dependent: :destroy
   has_many :broadcasts, dependent: :destroy
   has_many :zoom_meetings, dependent: :destroy
-  has_many :task_requests, dependent: :destroy
 
   accepts_nested_attributes_for :project_memberships
 
@@ -118,7 +117,7 @@ class Project < ApplicationRecord
     current_zoom_meeting = zoom_meetings.active.first
 
     unless current_zoom_meeting.present?
-      zoom_user = ZoomUser.free.first || ZoomUser.create
+      zoom_user = ZoomUser.for_new_meeting
       current_zoom_meeting = ZoomMeeting.create(zoom_user: zoom_user, project: self)
     end
 

app/models/task_request.rb

@@ -1,8 +0,0 @@
-class TaskRequest < ApplicationRecord
-  belongs_to :project
-  has_many_attached :files
-
-  enum status: [:pending, :completed, :cancelled]
-
-  scope :order_by_recent, -> { order(created_at: :desc) }
-end

app/models/zoom_meeting.rb

@@ -4,6 +4,9 @@ class ZoomMeeting < ApplicationRecord
   belongs_to :project, optional: true
   belongs_to :zoom_user
 
+  has_one_attached :recording
+  validates :recording, content_type: ['video/mp4']
+
   enum status: [:created, :started, :ended]
 
   after_create :create_api_meeting, if: -> { api_meeting_id.nil? }

app/models/zoom_user.rb

@@ -1,18 +1,28 @@
+require 'zoom_gateway'
+require 'securerandom'
 class ZoomUser < ApplicationRecord
   has_many :zoom_meetings, dependent: :nullify
 
+  enum tier: [:basic, :pro]
+
   after_create :create_api_user, if: -> { api_id.nil? }
   before_destroy :abort_destroy
 
+  scope :current_account, -> { where(account_number: ZoomGateway.ACCOUNT_NUMBER) }
   scope :free, -> { where.not(id: ZoomMeeting.active.pluck(:zoom_user_id)) }
 
-  def api_email
-    self.class.api_email(self.id)
-  end
-
   def create_api_user
-    self.api_id = gateway.create_host(api_email)
+    retries ||= 0
+    self.api_id = gateway.create_host(self.class.generate_api_email)
+    self.account_number = ZoomGateway.ACCOUNT_NUMBER
     save
+  rescue ZoomGateway::UserAlreadyExists => e
+    retries += 1
+    if retries < 3
+      retry # new api email will be generated automatically
+    else
+      raise e
+    end
   end
 
   def delete_api_user
@@ -21,9 +31,22 @@ class ZoomUser < ApplicationRecord
     save
   end
 
+  def current_account?
+    account_number == ZoomGateway.ACCOUNT_NUMBER
+  end
+
+  # Static methods
   class << self
-    def api_email(id)
+    def generate_api_email
-      "host#{id}@directme"
+      "host#{SecureRandom.random_number(10000)}@directme"
+    end
+
+    def for_new_meeting
+      user = public_send(ZoomGateway.USER_TYPE_NAME).current_account.free.first
+      if user.nil?
+        user = public_send(ZoomGateway.USER_TYPE_NAME).current_account.create
+      end
+      user
     end
   end
 

app/policies/contract_template_policy.rb

@@ -3,10 +3,8 @@ class ContractTemplatePolicy < ApplicationPolicy
     def resolve
       if user.account_manager?
         scope.left_outer_joins(:project).where(projects: {account: user.account})
-      elsif user.manager?
-        scope.left_outer_joins(project: :project_memberships).where(project_memberships: { user_id: user.id })
       else
-        scope.none
+        scope.left_outer_joins(project: :project_memberships).where(project_memberships: { user_id: user.id })
       end
     end
   end
@@ -14,8 +12,9 @@ class ContractTemplatePolicy < ApplicationPolicy
   def create?
     user.manager? || user.account_manager?
   end
+
   def show?
-    create?
+    true
   end
 
   def destroy?

app/policies/project_policy.rb

@@ -36,8 +36,4 @@ class ProjectPolicy < ApplicationPolicy
   def show_downloads?
     show?
   end
-
-  def show_task_results?
-    show?
-  end
 end

app/policies/task_request_policy.rb

@@ -1,25 +0,0 @@
-class TaskRequestPolicy < ApplicationPolicy
-  def index?
-    true
-  end
-
-  def show?
-    true
-  end
-
-  def create?
-    true
-  end
-
-  def destroy?
-    true
-  end
-
-  def update?
-    true
-  end
-
-  def cancel?
-    true
-  end
-end

app/telemetry/analytics.rb

@@ -152,24 +152,6 @@ class Analytics
     end
   end
   
-  def track_create_task_request(user_agent:, user_ip:)
-    if analytics_enabled?
-      identify
-      track(
-        {
-          user_id: user.id,
-          event: "Task request created",
-          properties: {
-            account: account.try(:name),
-            account_id: account.try(:id),
-            user_agent: user_agent,
-            ip: user_ip,
-          },
-        }
-      )
-    end
-  end
-  
   private
 
   attr_reader :user, :account, :backend

app/views/acquired_media_releases/_form.html.erb

@@ -4,7 +4,7 @@
     <div class="form-row">
       <%= form.text_field :name, required: true, wrapper_class: "col-12" %>
     </div>
-    <%= form.form_group :categories, label: { text: "Categories" } do %>
+    <%= form.form_group :categories, label: { text: "Licensed property type" } do %>
       <% AcquiredMediaRelease::CATEGORIES.each do |category| %>
         <%= form.check_box :categories, { multiple: true, label: category }, category, false %>
       <% end %>

app/views/admin/application/_side_nav.html.erb

@@ -7,9 +7,6 @@
       <li class="nav-item">
         <%= link_to fa_icon("users fw", text: "Users"), [:admin, :users], class: class_string("nav-link", "active" => controller_name == "users") %>
       </li>
-      <li class="nav-item">
-        <%= link_to fa_icon("tasks fw", text: "Task Requests"), [:admin, :task_requests], class: class_string("nav-link", "active" => controller_name == "task_requests") %>
-      </li>
       <li class="nav-item">
         <%= link_to fa_icon("bug fw", text: "Errors"), "https://sentry.io/bigmedia/", class: "nav-link", target: :_blank %>
       </li>

app/views/admin/task_requests/_form.html.erb

@@ -1,13 +0,0 @@
-<%= errors_summary_for task_request %>
-
-<%= bootstrap_form_with model: model, local: true do |form| %>
-  <%= form.select :status, options_for_select(TaskRequest.statuses.except(:cancelled).keys, task_request.status), { class: "form-control" } %>
-  <%= form.text_field :deliverable_url  %>
-
-  <div class="row align-items-center text-center mt-4">
-    <%= link_to t("shared.cancel"), [:admin, :task_requests], class: "col-3 text-reset" %>
-    <div class="col-9">
-      <%= form.submit class: class_string("btn btn-block", ["btn-success", "btn-primary"] => task_request.new_record?), data: { disable_with: t("shared.disable_with") } %>
-    </div>
-  </div>
-<% end %>

app/views/admin/task_requests/_task_request.html.erb

@@ -1,31 +0,0 @@
-<tr id="<%= dom_id(task_request) %>">
-  <td>
-    <%= task_request.id %>
-  </td>
-  <td>
-    <%= task_request.project.account.name %>
-  </td>
-  <td>
-    <%= task_request.project.name %>
-  </td>
-  <td>
-    <%= task_request.created_at.strftime("%D") %>
-  </td>
-  <td>
-    <%= task_request.deadline.try(:strftime, '%D') %>
-  </td>
-  <td>
-    <%= task_request.time_allowed %>
-  </td>
-  <td>
-    <%= task_request.status.titleize %>
-  </td>
-  <td class="text-right">
-    <div class="btn-group">
-      <%= button_tag "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">
-        <%= link_to fa_icon("pencil", text: "Edit"), [:edit, :admin, task_request], class: "dropdown-item" %>
-      </div>
-    </div>
-  </td>
-</tr>

app/views/admin/task_requests/edit.html.erb

@@ -1,6 +0,0 @@
-<div class="card shadow-sm">
-  <%= card_header text: "Edit Task Request", close_action_path: [:admin, :task_requests] %>
-  <div class="card-body">
-    <%= render "form", model: [:admin, @task_request], task_request: @task_request %>
-  </div>
-</div>

app/views/admin/task_requests/index.html.erb

@@ -1,25 +0,0 @@
-<div class="border bg-white rounded shadow-sm pb-3 table-responsive">
-  <table class="table table-striped tr-px-4 align-all-middle">
-    <thead class="thead-light">
-      <tr>
-        <th>Task ID</th>
-        <th>Account Name</th>
-        <th>Project Name</th>
-        <th>Created On</th>
-        <th>Deadline</th>
-        <th>Time Allowed</th>
-        <th>Status</th>
-        <th></th>
-      </tr>
-    </thead>
-    <tbody id="users">
-      <% if @task_requests.any? %>
-        <%= render @task_requests %>
-      <% else %>
-        <tr>
-          <td colspan="20" class="py-4 text-center text-muted"><%= t(".empty") %></td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-</div>

app/views/appearance_releases/_appearance_release.html.erb

@@ -2,7 +2,7 @@
   <td data-behavior="select"><%= check_box_tag "appearance_release_ids[]", appearance_release.id, false %></td>
   <td>
     <% if appearance_release.photo.attached? %>
-      <%= image_tag medium_variant(appearance_release.photo), class: "img-fluid" %>
+      <%= image_tag medium_variant(appearance_release.photo) %>
     <% else %>
       <%= fa_icon("warning", text: t(".no_photos"), class: "text-danger") %>
     <% end %>
@@ -11,10 +11,11 @@
     <%= appearance_release.name %>
   </td>
   <td>
-    <%= number_to_phone appearance_release.person_phone %>
+    <%= contact_info( 
-  </td>
+      address: appearance_release.person_address, 
-  <td>
+      phone: appearance_release.person_phone, 
-    <%= mail_to appearance_release.person_email %>
+      email: appearance_release.person_email
+    ) %>
   </td>
   <td>
     <%= notes_preview appearance_release.notes.order_by_recent %>

app/views/appearance_releases/_form.html.erb

@@ -52,7 +52,7 @@
       <% end %>
       <div class="d-inline-block">
         <%= form.hidden_field :person_photo, value: form.object.person_photo.signed_id if appearance_release.person_photo.attached?%>
-        <%= form.file_field :person_photo, wrapper_class: "required", hide_label: true, required: !appearance_release.person_photo.attached?, data: { ujs_target: "person-photo-input" }, help: "PNG or JPG only", accept: appearance_release.class.face_photo_acceptable_content_types.join(",") %>
+        <%= form.file_field :person_photo, hide_label: true, data: { ujs_target: "person-photo-input" }, help: "PNG or JPG only", accept: appearance_release.class.face_photo_acceptable_content_types.join(",") %>
       </div>
     </div>
 

app/views/appearance_releases/index.html.erb

@@ -46,8 +46,7 @@
         <th data-behavior="all-selectable"><%= check_box_tag "appearance_release_ids[]", false, false %></th>
         <th></th>
         <th><%= AppearanceRelease.human_attribute_name(:person_name) %></th>
-        <th><%= AppearanceRelease.human_attribute_name(:person_phone) %></th>
+        <th><%= AppearanceRelease.human_attribute_name(:contact_info) %></th>
-        <th><%= AppearanceRelease.human_attribute_name(:person_email) %></th>
         <th><%= t(".table_headers.notes") %></th>
         <th><%= t(".table_headers.tags") %></th>
         <th><%= t(".table_headers.signed_at") %></th>

app/views/application/_sidebar.html.erb

@@ -35,12 +35,6 @@
           <%= product_wordmark :deliver_me, class: class_string("d-inline-block", "disabled" => !Current.account.deliverme_enabled?) %>
         <% end %>
       </li>
-      <li class="nav-item">
-        <%= link_to [project, :task_requests], class: class_string("nav-link", "active" => controller_name == "task_requests") do %>
-          <%= lock_icon_for(Current.account, :assistme) %>
-          <%= product_wordmark :assist_me, class: class_string("d-inline-block", "disabled" => !Current.account.assistme_enabled?) %>
-        <% end %>
-      </li>
     </ul>
   </nav>
   <hr class="divider-light mx-n4">

app/views/broadcasts/_file.html.erb

@@ -1,4 +1,4 @@
-<li class="my-2">
+<li class="my-2" id="<%= dom_id(file) %>">
   <% if file.variable? %>
     <%= link_to image_tag(file.variant(resize_and_pad: [300, 300, background: "#F7F8F9"]), class: "bg-light img-thumbnail img-fluid"), file, target: "_blank" %>
   <% else %>

app/views/broadcasts/_file_form.html.erb

@@ -1,4 +1,4 @@
 <%= bootstrap_form_for model, layout: :inline, remote: true do |form| %>
-  <%= form.file_field :files, direct_upload: true, multiple: true, accept: "*", hide_label: true, wrapper_class: "w-65 mr-2" %>
+  <%= form.file_field :files, direct_upload: true, multiple: true, accept: "*", hide_label: true, wrapper_class: "w-65 mr-2", id: "broadcast_files_#{token}" %>
   <%= form.button fa_icon("upload", text: "Add File"), class: "btn btn-primary", type: :submit, data: { disable_with: fa_icon("spinner", text: "Adding File") } %>
 <% end %>

app/views/broadcasts/_files_section.html.erb

@@ -0,0 +1,21 @@
+<div class="text-center pb-2" id="broadcast_file_form_<%= broadcast.token %>">
+  <% if controller.class.module_parent.to_s == "Public" %>
+    <%= render partial: "public/broadcasts/file_form", locals: { model: [broadcast], token: broadcast.token } %>
+  <% else %>
+    <%= render partial: "broadcasts/file_form", locals: { model: [broadcast.project, broadcast], token: broadcast.token } %>
+  <% end %>
+</div>
+<div class="overflow-auto mh-30">
+  <ul class="list-unstyled d-flex flex-column align-items-center text-center" id="broadcast_file_list_<%= broadcast.token %>">
+    <% if files.present? %>
+      <%= render partial: "broadcasts/file", collection: files %>
+    <% else %>
+      <li class="my-3">
+        Files will appear here.
+      </li>
+    <% end %>
+  </ul>
+  <div class="d-flex mt-2 justify-content-center" id="broadcast_files_pagination_<%= broadcast.token %>">
+    <%= will_paginate(files, param_name: 'files_page', params: { active_files_tab: broadcast.token }) if files.present? %>
+  </div>
+</div>

app/views/broadcasts/show.html.erb

@@ -5,6 +5,12 @@
   <% if @broadcast %>
     <meta name="broadcast-token" content="<%= @broadcast.token %>">
   <% end %>
+  <% # Subscribe to Action Cable for every broadcast including those in multi-view %>
+  <% @multi_view_broadcasts.each do |multi_view_broadcast| %>
+    <% if multi_view_broadcast.token != @broadcast.token %>
+      <meta name="broadcast-token" content="<%= multi_view_broadcast.token %>">
+    <% end %>
+  <% end %>
 <% end %>
 
 <% content_for :header do %>
@@ -24,9 +30,7 @@
         <div class="d-flex justify-content-between align-items-center">
           <h1 class="h3 m-0"><%= @broadcast.name %></h1>
           <div class="dropdown">
-            <a class="btn btn-light border dropdown-toggle" href="#" role="button" id="dropdownMenuLink" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+            <%= link_to "Switch View", "#", class: "btn btn-light border dropdown-toggle", role: "button", id: "dropdownMenuLink", data: { toggle: "dropdown" }, aria: { haspopup: "true", expanded: "false" } %>
-              Switch View
-            </a>
             <div class="dropdown-menu" aria-labelledby="dropdownMenuLink">
               <h5 class="dropdown-header">Live Streams</h5>
               <span class="dropdown-item active"><%= fa_icon("check", text: @broadcast.name.titleize) %></span>
@@ -72,13 +76,10 @@
       <div class="card-header">
         <ul class="nav nav-tabs card-header-tabs">
           <li class="nav-item">
-            <a class="<%= class_string("nav-link", "active" => !params[:active_tab].present?) %>" href="#home" data-toggle="tab">Home</a>
+            <%= link_to "Home", "#home", class: class_string("nav-link", "active" => !params[:active_tab].present?), data: { toggle: "tab" } %>
           </li>
           <li class="nav-item">
-            <a class="<%= class_string("nav-link", "active" => params[:active_tab] == "files") %>" href="#files" data-toggle="tab">Files</a>
+            <%= link_to "Previous Sessions", "#recordings", class: class_string("nav-link", "active" => params[:active_tab] == "recordings"), data: { toggle: "tab" } %>
-          </li>
-          <li class="nav-item">
-            <a class="<%= class_string("nav-link", "active" => params[:active_tab] == "recordings") %>" href="#recordings" data-toggle="tab">Previous Sessions</a>
           </li>
         </ul>
       </div>
@@ -118,30 +119,6 @@
             <p class="card-text">If you want to join the ZOOM meeting dedicated to this broadcast, follow the link below.</p>
             <%= link_to 'Video Conference', @conference_url, class: 'btn btn-primary btn-block', target: '_blank' %>
           </div>
-          <div class="<%= class_string("tab-pane fade show", "active" => params[:active_tab] == 'files') %>" id="files">
-            <div class="text-center pb-2" id="broadcast_file_form">
-              <% if controller.class.module_parent.to_s == "Public" %>
-                <%= render partial: "public/broadcasts/file_form", locals: { model: [@broadcast], token: @broadcast.token } %>
-              <% else %>
-                <%= render partial: "broadcasts/file_form", locals: { model: [@project, @broadcast] } %>
-              <% end %>
-            </div>
-            <p class="alert alert-info mt-2"><%= fa_icon("warning", text: "You may need to refresh the page to see new files uploaded by other team members") %></p>
-            <div class="overflow-auto mh-30">
-              <ul class="list-unstyled d-flex flex-column align-items-center text-center" id="broadcast_file_list">
-                <% if @files.present? %>
-                  <%= render partial: "broadcasts/file", collection: @files %>
-                <% else %>
-                  <li class="my-3">
-                    Files will appear here.
-                  </li>
-                <% end %>
-              </ul>
-              <div class="d-flex mt-2 justify-content-center" id="broadcast_files_pagination">
-                <%= will_paginate(@files, params: { active_tab: 'files' })  if @files.present? %>
-              </div>
-            </div>
-          </div>
           <div class="<%= class_string("tab-pane fade show", "active" => params[:active_tab] == 'recordings') %>" id="recordings">
             <div id="broadcast_recordings">
               <%= render partial: 'broadcasts/broadcast_recordings', locals: { recordings: @recordings, broadcast: @broadcast } %>
@@ -150,5 +127,36 @@
         </div>
       </div>
     </div>
+
+    <!-- files section -->
+    <div id="files" class="card shadow-sm mb-3">
+      <div class="card-header">
+        <h2 class="h5">Files</h2>
+        <% if @multi_view_broadcasts %>
+          <ul class="nav nav-tabs card-header-tabs">
+            <% @multi_view_broadcasts.each_with_index do |mvb, index| %>
+              <li class="nav-item">
+                <%= link_to mvb.name, "#files_broadcast_#{mvb.token}", class: class_string("nav-link", "active" => (params[:active_files_tab] == mvb.token || (params[:active_files_tab].nil? && index == 0))), data: { toggle: "tab" } %>
+              </li>
+            <% end %>
+          </ul>
+        <% end %>
+      </div>
+      <div class="card-body p-3">
+        <div class="tab-content">
+          <% if @multi_view_broadcasts.present? %>
+            <% @multi_view_broadcasts.each_with_index do |mvb, index| %>
+              <div class="<%= class_string("tab-pane fade show", "active" => (params[:active_files_tab] == mvb.token || (params[:active_files_tab].nil? && index == 0))) %>" id="files_broadcast_<%= mvb.token %>">
+                <%= render partial: 'broadcasts/files_section', locals: { broadcast: mvb, files: mvb.files } %>
+              </div>
+            <% end %>
+          <% else %>
+            <div class="tab-pane fade show active" id="files_broadcast_<%= @broadcast.id %>">
+              <%= render partial: 'broadcasts/files_section', locals: { broadcast: @broadcast, files: @files } %>
+            </div>
+          <% end %>
+        </div>
+      </div>
+    </div>
   </div>
 <div>

app/views/broadcasts/update.js.erb

@@ -1,5 +1,9 @@
-$("#broadcast_file_form").html("<%= j render(partial: "broadcasts/file_form", locals: { model: [@project, @broadcast] }) %>");
+$("#broadcast_file_form_<%= @broadcast.token %>").html("<%= j render(partial: "broadcasts/file_form", locals: { model: [@project, @broadcast], token: @broadcast.token }) %>");
-$("#broadcast_file_list").html("<%= j render(partial: "broadcasts/file", collection: @files) %>");
+
-$("#broadcast_files_pagination").html("<%= j will_paginate(@files) %>");
+var file_id = "#<%= dom_id(@files.first) %>"
+if ($("#broadcast_file_list_<%= @broadcast.token %>").has(file_id).length == 0) {
+  $("#broadcast_file_list_<%= @broadcast.token %>").html("<%= j render(partial: "broadcasts/file", collection: @files) %>");
+  $("#broadcast_files_pagination_<%= @broadcast.token %>").html("<%= j will_paginate(@files, param_name: 'files_page', params: { active_files_tab: @broadcast.token }) %>");
+}
 
 bsCustomFileInput.init();

app/views/contracts/_signature_page.html.erb

@@ -29,6 +29,7 @@
   <% if releasable.model_name == "LocationRelease" %>
     <%= description_list_pair "Filming Started On:", releasable&.filming_started_on&.strftime("%D") %>
     <%= description_list_pair "Filming Ended On:", releasable&.filming_ended_on&.strftime("%D") %>
+    <%= description_list_pair "Filming Hours:", releasable&.filming_hours %>
   <% end %>
   <% if contract_template.fee? %>
     <%= description_list_pair "Fee:", number_to_currency(contract_template.fee) %>

app/views/location_releases/_form.html.erb

@@ -11,14 +11,10 @@
 
   <%= field_set_tag content_tag(:span, t(".signer_details.heading"), class: "h6 text-muted text-uppercase") do %>
     <div class="form-row">
-      <%= form.text_field :person_first_name, wrapper_class: "col-sm-3" %>
+      <%= form.text_field :person_first_name, wrapper_class: "col-sm-6" %>
-      <%= form.text_field :person_last_name, wrapper_class: "col-sm-3" %>
+      <%= form.text_field :person_last_name, wrapper_class: "col-sm-6" %>
       <%= form.phone_field :person_phone, wrapper_class: "col-sm-6" %>
-    </div>
-    <div class="form-row">
       <%= form.email_field :person_email, wrapper_class: "col-sm-6" %>
-    </div>
-    <div class="form-row">
       <%= form.text_field :person_company, wrapper_class: "col-sm-6" %>
       <%= form.text_field :person_title, wrapper_class: "col-sm-6" %>
     </div>
@@ -38,6 +34,7 @@
     <div class="form-row">
       <%= form.text_field :filming_started_on, wrapper_class: "col-sm-6", class: "datepicker-control", readonly: true  %>
       <%= form.text_field :filming_ended_on, wrapper_class: "col-sm-6", class: "datepicker-control", readonly: true %>
+      <%= form.text_field :filming_hours, wrapper_class: "col-sm-12" %>
     </div>
   <% end %>
 

app/views/material_releases/_form.html.erb

@@ -13,14 +13,10 @@
 
   <%= field_set_tag content_tag(:span, t(".signer_details.heading"), class: "h6 text-muted text-uppercase") do %>
     <div class="form-row">
-      <%= form.text_field :person_first_name, wrapper_class: "col-sm-3" %>
+      <%= form.text_field :person_first_name, wrapper_class: "col-sm-6" %>
-      <%= form.text_field :person_last_name, wrapper_class: "col-sm-3" %>
+      <%= form.text_field :person_last_name, wrapper_class: "col-sm-6" %>
       <%= form.phone_field :person_phone, wrapper_class: "col-sm-6" %>
-    </div>
-    <div class="form-row">
       <%= form.email_field :person_email, wrapper_class: "col-sm-6" %>
-    </div>
-    <div class="form-row">
       <%= form.text_field :person_company, wrapper_class: "col-sm-6" %>
       <%= form.text_field :person_title, wrapper_class: "col-sm-6" %>
     </div>

app/views/projects/show.html.erb

@@ -16,11 +16,6 @@
       <%= link_to t("projects.show.downloads"), [@project, :downloads], class: "text-decoration-none text-reset stretched-link" %>
     <% end %>
   <% end %>
-  <% if policy(Project).show_task_results? %>
-    <%= render "folder_card" do %>
-      <%= link_to t("projects.show.task_requests"), [@project, :task_requests, completed_only: true], class: "text-decoration-none text-reset stretched-link" %>
-    <% end %>
-  <% end %>
 </div>
 
 <hr/>

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

@@ -15,12 +15,12 @@
 
       <%= card_field_set_tag t(".acquired_media_info.heading") do %>
         <div class="form-row">
-          <%= form.text_field :name, required: true, wrapper_class: "col-12", label: 'Licensor ("Owner")' %>
+          <%= form.text_field :name, required: true, wrapper_class: "col-12" %>
         </div>
         <div class="form-row">
           <%= form.text_area :description, wrapper_class: "col-12" %>
         </div>
-        <%= form.form_group :categories, label: { text: "Categories" } do %>
+        <%= form.form_group :categories, label: { text: "Licensed property type" } do %>
           <% AcquiredMediaRelease::CATEGORIES.each do |category| %>
             <%= form.check_box :categories, { multiple: true, label: category }, category, false %>
           <% end %>
@@ -31,8 +31,11 @@
 
       <%= card_field_set_tag t(".personal_info.heading") do %>
         <div class="form-row">
+          <%= form.text_field :person_first_name, wrapper_class: "col-sm-6" %>
+          <%= form.text_field :person_last_name, wrapper_class: "col-sm-6" %>
           <%= form.text_field :person_title, wrapper_class: "col-sm-6" %>
-          <%= form.text_field :person_phone, wrapper_class: "col-sm-6", label: 'Phone' %>
+          <%= form.phone_field :person_phone, wrapper_class: "col-sm-6", label: 'Phone' %>
+          <%= form.email_field :person_email, wrapper_class: "col-sm-6", label: 'Email' %>
           <%= form.text_field :person_fax, wrapper_class: "col-sm-6", label: 'Fax' %>
         </div>
         <%= render "shared/address_fields", form: form, subject: "person" %>

app/views/public/broadcasts/_file_form.html.erb

@@ -1,4 +1,4 @@
 <%= bootstrap_form_for model, url: broadcast_url(token: token), layout: :inline, remote: true do |form| %>
-  <%= form.file_field :files, direct_upload: true, multiple: true, accept: "*", hide_label: true, wrapper_class: "w-65 mr-2" %>
+  <%= form.file_field :files, direct_upload: true, multiple: true, accept: "*", hide_label: true, wrapper_class: "w-65 mr-2", id: "broadcast_files_#{token}" %>
   <%= form.button fa_icon("upload", text: "Add File"), class: "btn btn-primary", type: :submit, data: { disable_with: fa_icon("spinner", text: "Adding File") } %>
 <% end %>

app/views/public/broadcasts/update.js.erb

@@ -1,5 +1,9 @@
-$("#broadcast_file_form").html("<%= j render(partial: "public/broadcasts/file_form", locals: { model: [@project, @broadcast], token: @broadcast.token }) %>");
+$("#broadcast_file_form_<%= @broadcast.token %>").html("<%= j render(partial: "public/broadcasts/file_form", locals: { model: [@project, @broadcast], token: @broadcast.token }) %>");
-$("#broadcast_file_list").html("<%= j render(partial: "broadcasts/file", collection: @files) %>");
+
-$("#broadcast_files_pagination").html("<%= j will_paginate(@files) %>");
+var file_id = "#<%= dom_id(@files.first) %>"
+if ($("#broadcast_file_list_<%= @broadcast.token %>").has(file_id).length == 0) {
+  $("#broadcast_file_list_<%= @broadcast.token %>").html("<%= j render(partial: "broadcasts/file", collection: @files) %>");
+  $("#broadcast_files_pagination_<%= @broadcast.token %>").html("<%= j will_paginate(@files) %>");
+}
 
 bsCustomFileInput.init();

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

@@ -27,11 +27,7 @@
           <%= form.text_field :person_first_name, wrapper_class: "col-sm-6" %>
           <%= form.text_field :person_last_name, wrapper_class: "col-sm-6" %>
           <%= form.phone_field :person_phone, wrapper_class: "col-sm-6" %>
-        </div>
-        <div class="form-row">
           <%= form.email_field :person_email, wrapper_class: "col-sm-6" %>
-        </div>
-        <div class="form-row">
           <%= form.text_field :person_company, wrapper_class: "col-sm-6" %>
           <%= form.text_field :person_title, wrapper_class: "col-sm-6" %>
         </div>
@@ -45,6 +41,7 @@
         <div class="form-row">
           <%= form.text_field :filming_started_on, wrapper_class: "col-sm-6", class: "datepicker-control", readonly: true %>
           <%= form.text_field :filming_ended_on, wrapper_class: "col-sm-6", class: "datepicker-control", readonly: true %>
+          <%= form.text_field :filming_hours, wrapper_class: "col-sm-12" %>
         </div>
       <% end %>
 

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

@@ -27,11 +27,7 @@
           <%= form.text_field :person_first_name, wrapper_class: "col-sm-6" %>
           <%= form.text_field :person_last_name, wrapper_class: "col-sm-6" %>
           <%= form.phone_field :person_phone, wrapper_class: "col-sm-6" %>
-        </div>
-        <div class="form-row">
           <%= form.email_field :person_email, wrapper_class: "col-sm-6" %>
-        </div>
-        <div class="form-row">
           <%= form.text_field :person_company, wrapper_class: "col-sm-6" %>
           <%= form.text_field :person_title, wrapper_class: "col-sm-6" %>
         </div>

app/views/shared/_address_fields.html.erb

@@ -11,6 +11,6 @@
 </div>
 <%= form.form_group "#{field_name_prefix}address_country" do %>
   <%= form.label "#{field_name_prefix}address_country" %>
-  <%= form.country_select "#{field_name_prefix}address_country", { priority: %w(US CA), prompt: true }, class: "form-control custom-select" %>
+  <%= form.country_select "#{field_name_prefix}address_country", { selected: 'US', priority: %w(US CA), prompt: true }, class: "form-control custom-select" %>
 <% end %>
 

app/views/task_requests/_form.html.erb

@@ -1,34 +0,0 @@
-<%= errors_summary_for task_request %>
-
-<%= bootstrap_form_with model: model, local: true do |form| %>
-  <%= form.text_area :description %>
-  <%= form.text_field :deadline, class: "datepicker-control"  %>
-  <%= form.text_field :time_allowed  %>
-  <%= form.text_area :additional_notes %>
-  <%= field_set_tag content_tag(:span, "Files", class: "h6 text-muted text-uppercase") do %>
-    <div class="field d-none">
-      <%= form.label :files %>
-      <%= form.file_field :files, disable: true, direct_upload: true, multiple: true, id: "task_request_files", hide_label: true %>
-      <% task_request.files.each do |file| %>
-        <% unless file.persisted? %>
-          <%= hidden_field_tag "#{task_request.model_name.param_key}[files][]", file.signed_id %>
-        <% end %>
-      <% end %>
-    </div>
-
-    <div class="dropzone field border-dashed" 
-      data-accepted-files="audio/*,image/*,video/*,application/*"
-      data-behavior="dropzone" 
-      data-file-input-id="task_request_files" 
-      data-existing-files="<%= mock_photos_json(task_request.files) %>" 
-      data-placeholder="<%= dropzone_placeholder_message_for(task_request) %>"
-      data-submit-button="#submit_folder"></div>
-  <% end %>
-
-  <div class="row align-items-center text-center mt-4">
-    <%= link_to t("shared.cancel"), [project, :task_requests], class: "col-3 text-reset" %>
-    <div class="col-9">
-      <%= form.submit class: class_string("btn btn-block", ["btn-success", "btn-primary"] => task_request.new_record?), data: { disable_with: t("shared.disable_with") } %>
-    </div>
-  </div>
-<% end %>

app/views/task_requests/_task_request.html.erb

@@ -1,35 +0,0 @@
-<tr>
-  <td>
-    <%= task_request.created_at.strftime('%D') %>
-  </td>
-  <td>
-    <%= task_request.deadline.try(:strftime, '%D') %>
-  </td>
-  <td>
-    <%= task_request.time_allowed %>
-  </td>
-  <td>
-    <%= task_request.status.titleize %>
-  </td>
-  <% if params[:completed_only] %>
-    <td>
-      <%= task_request.deliverable_url %>
-    </td>
-  <% end %>
-  <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">
-        <% if policy(task_request).show? %>
-          <%= link_to fa_icon("tasks fw", text: "View"), [task_request.project, task_request], class: "dropdown-item", target: '_blank' %>
-        <% end %>
-        <% if policy(task_request).edit? %>
-          <%= link_to fa_icon("pencil fw", text: "Edit"), [:edit, task_request.project, task_request], class: "dropdown-item" %>
-        <% end %>
-        <% if policy(task_request).cancel? && !task_request.cancelled? %>
-          <%= link_to fa_icon("ban fw", text: "Cancel"), [:cancel, task_request.project, task_request], class: "dropdown-item", method: :post %>
-        <% end %>
-      </div>
-    </div>
-  </td>
-</tr>

app/views/task_requests/edit.html.erb

@@ -1,6 +0,0 @@
-<div class="card shadow-sm">
-  <%= card_header text: t(".heading"), close_action_path: [@project, :task_requests] %>
-  <div class="card-body">
-    <%= render "form", model: [@project, @task_request], task_request: @task_request, project: @project %>
-  </div>
-</div>

app/views/task_requests/index.html.erb

@@ -1,41 +0,0 @@
-<%= product_wordmark :assist_me, class: "small mb-3" %>
-
-<div class="row">
-  <div class="col-md-12">
-    <div class="d-md-flex d-sm-flex flex-sm-column flex-md-row flex-md-wrap mb-3">
-      <% if policy(TaskRequest).new? %>
-        <%= link_to fa_icon("plus", text: t(".actions.new")), [:new, @project, :task_request], class: "btn btn-primary mb-2" %>
-      <% end %>
-    </div>
-  </div>
-</div>
-
-<div class="border bg-white rounded shadow-sm pb-3 table-responsive">
-  <table class="table table-striped tr-px-4 align-all-middle">
-    <thead class="thead-light">
-      <tr>
-        <th><%= t(".table_headers.task_request_created_on") %></th>
-        <th><%= t(".table_headers.task_request_deadline") %></th>
-        <th><%= t(".table_headers.task_request_time_allowed") %></th>
-        <th><%= t(".table_headers.task_request_status") %></th>
-        <% if params[:completed_only] %>
-          <th><%= t(".table_headers.task_request_results") %></th>
-        <% end %>
-        <th></th>
-      </tr>
-    </thead>
-    <tbody id="task_requests">
-      <% if @task_requests.any? %>
-        <%= render @task_requests %>
-      <% else %>
-        <tr>
-          <td colspan="5" class="py-4 text-center text-muted"><%= t(".empty") %></td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-</div>
-
-<div id="task_requests_pagination" class="mt-3">
-  <%= will_paginate @task_requests %>
-</div>

app/views/task_requests/new.html.erb

@@ -1,6 +0,0 @@
-<div class="card shadow-sm">
-  <%= card_header text: t(".heading"), close_action_path: [@project, :task_requests] %>
-  <div class="card-body">
-    <%= render "form", model: [@project, @task_request], task_request: @task_request, project: @project %>
-  </div>
-</div>

app/views/task_requests/show.html.erb

@@ -1,18 +0,0 @@
-<dl>
-  <%= description_list_pair_for @task_request, :description, append: ":" %>
-  <%= description_list_pair_for @task_request, :created_at, append: ":" %>
-  <%= description_list_pair_for @task_request, :deadline, append: ":" %>
-  <%= description_list_pair_for @task_request, :time_allowed, append: ":" %>
-  <%= description_list_pair_for @task_request, :additional_notes, append: ":" %>
-  <%= description_list_pair_for @task_request, :status, append: ":" %>
-  <dt>Files:</dt>
-  <dd>
-    <% if @task_request.files.present? %>
-      <% @task_request.files.each do |file| %>
-        <%= file.filename %><br>
-      <% end %>
-    <% else %>
-      "No files attached."
-    <% end %>
-  </dd>
-</dl>

config/locales/en.yml

@@ -93,11 +93,6 @@ en:
     application:
       header:
         sign_out: Sign Out
-    task_requests:
-      index:
-        empty: Task requests will appear here
-      update:
-        notice: The task request has been updated successfully
     users:
       create:
         notice: The user was created
@@ -310,6 +305,8 @@ en:
         graphics_only_edl_file: If you do not upload a Graphics Only EDL, the software will not generate a Graphics Cue List.
     label:
       acquired_media_release:
+        description: Description of licensed property
+        name: Name of licensed property 
         person_address: Address
         person_address_city: City
         person_address_country: Country
@@ -319,6 +316,8 @@ en:
         person_address_zip: Zip code 
         person_company: Company
         person_email: Email address
+        person_first_name: Licensor/Owner first name
+        person_last_name: Licensor/Owner last name
         person_name: Name
         person_phone: Phone number
         person_title: Title
@@ -332,16 +331,23 @@ en:
         address_city: City
         address_country: Country
         address_state: State
-        address_street1: Address 
+        address_street1: Address of Property
         address_street2: Address (Line 2)
         address_zip: Zip code 
+        filming_ended_on: Filming end date
+        filming_started_on: Filming start date
+        name: Name of property
         person_address_city: City
         person_address_country: Country
         person_address_state: State
         person_address_street1: Address 
         person_address_street2: Address (Line 2)
         person_address_zip: Zip code 
+        person_first_name: Owner first name
+        person_last_name: Owner last name
       material_release:
+        description: Description of licensed material
+        name: Name of licensed material
         person_address: Address
         person_address_city: City
         person_address_country: Country
@@ -351,6 +357,8 @@ en:
         person_address_zip: Zip code 
         person_company: Company
         person_email: Email address
+        person_first_name: Licensor/Owner first name
+        person_last_name: Licensor/Owner last name
         person_name: Name
         person_phone: Phone number
         person_title: Title
@@ -549,7 +557,7 @@ en:
       photos:
         heading: 4 of 4 Photos
       signer_details: 
-        heading: 2 of 4 Signer Details
+        heading: 2 of 4 Owner Details
     index:
       actions:
         new: Import Release
@@ -583,7 +591,7 @@ en:
       photos:
         heading: 4 of 4 Photos
       signer_details:
-        heading: 2 of 4 Signer Details
+        heading: 2 of 4 Licensor/Owner Details
     index:
       actions:
         new: Import Release
@@ -644,6 +652,7 @@ en:
     create:
       notice: Check your email for password reset instructions
     edit:
+      notice: This password reset URL has expired or is invalid. Try again by clicking "Forgot your password?" below
       submit: Save New Password
       title: Password Reset
     new:
@@ -734,7 +743,6 @@ en:
       music_release: Music Releases (%{count})
       report: Reports
       talent_release: Talent Releases (%{count})
-      task_requests: Tasks
   public:
     acquired_media_releases:
       new:
@@ -746,9 +754,7 @@ en:
         legal:
           heading: Legal
         personal_info:
-          heading: Person Details
+          heading: Licensor/Owner Contact Information
-        personal_info:
-          heading: Signer's Contact Information
         signature:
           heading: Signature
     appearance_releases:
@@ -787,7 +793,7 @@ en:
       new:
         cancel: Cancel
         contact_info:
-          heading: Signer Information
+          heading: Owner Contact Information
         filming_info:
           heading: Filming Information
         legal:
@@ -802,7 +808,7 @@ en:
       new:
         cancel: Cancel
         contact_info:
-          heading: Signer's Contact Information
+          heading: Licensor/Owner Contact Information
         legal:
           heading: Legal
         release_info:
@@ -875,7 +881,6 @@ en:
       title: Sign In
   shared:
     ago: ago
-    assist_me: Assist
     back: Back
     cancel: Cancel
     clear: Clear
@@ -936,31 +941,6 @@ en:
         manage: Manage
     update:
       notice: The talent release has been updated
-  task_requests:
-    cancel:
-      notice: Task has been cancelled successfully.
-    create:
-      notice: Task request created succussfully.
-    edit:
-      heading:
-        Edit Task Request
-    index:
-      actions:
-        new: Create Task Request
-      empty: Task requests will appear here.
-      table_headers:
-        task_request_created_on: Created On
-        task_request_deadline: Deadline
-        task_request_results: Task Results
-        task_request_status: Status
-        task_request_time_allowed: Time Allowed
-    new:
-      heading: New Task Request
-    task_request:
-      actions:
-        manage: Manage
-    update:
-      notice: Task request updated successfully.
   user_mailer: 
     existing_account: 
       subject: You've been added as a ME Suite Account Manager 

config/routes.rb

@@ -31,7 +31,6 @@ Rails.application.routes.draw do
       resources :users, only: [:index, :new, :create, :edit, :update, :destroy] do
         resource :masquerade, only: :create
       end
-      resources :task_requests, only: [:index, :edit, :update]
 
       root to: "accounts#index", as: :signed_in_root
     end
@@ -98,11 +97,6 @@ Rails.application.routes.draw do
           delete :destroy_file
         end
       end
-      resources :task_requests, except: :destroy do
-        member do
-          post :cancel
-        end
-      end
     end
     resource :profile, only: [:show, :update]
     resources :videos, only: [] do

db/data_migrations/20200526114957_assign_account_number_and_type_to_zoom_users.rb

@@ -0,0 +1,7 @@
+class AssignAccountNumberAndTypeToZoomUsers < ActiveRecord::DataMigration
+  def up
+    ZoomUser.find_each do |zu|
+      zu.update account_number: ENV.fetch('ZOOM_ACCOUNT_NUMBER'), tier: (ENV['ZOOM_USER_TYPE'] == 'pro' ? 1 : 0)
+    end
+  end
+end

db/migrate/20200518200245_create_task_requests.rb

@@ -1,14 +0,0 @@
-class CreateTaskRequests < ActiveRecord::Migration[6.0]
-  def change
-    create_table :task_requests do |t|
-      t.references :project
-      t.text :description
-      t.datetime :deadline
-      t.string :time_allowed
-      t.text :additional_notes
-      t.integer :status, default: 0
-      
-      t.timestamps
-    end
-  end
-end

db/migrate/20200519191908_add_deliverable_url_to_task_requests.rb

@@ -1,5 +0,0 @@
-class AddDeliverableUrlToTaskRequests < ActiveRecord::Migration[6.0]
-  def change
-    add_column :task_requests, :deliverable_url, :string
-  end
-end

db/migrate/20200526113516_add_account_number_and_type_to_zoom_users.rb

@@ -0,0 +1,6 @@
+class AddAccountNumberAndTypeToZoomUsers < ActiveRecord::Migration[6.0]
+  def change
+    add_column :zoom_users, :account_number, :string
+    add_column :zoom_users, :tier, :integer, default: 0
+  end
+end

db/migrate/20200603090419_add_filming_hours_to_location_releases.rb

@@ -0,0 +1,5 @@
+class AddFilmingHoursToLocationReleases < ActiveRecord::Migration[6.0]
+  def change
+    add_column :location_releases, :filming_hours, :text
+  end
+end

db/structure.sql

@@ -44,7 +44,7 @@ COMMENT ON EXTENSION pg_trgm IS 'text similarity measurement and index searching
 CREATE FUNCTION public.pg_search_dmetaphone(text) RETURNS text
     LANGUAGE sql IMMUTABLE STRICT
     AS $_$
-    SELECT array_to_string(ARRAY(SELECT dmetaphone(unnest(regexp_split_to_array($1, E'\\s+')))), ' ')	
+  SELECT array_to_string(ARRAY(SELECT dmetaphone(unnest(regexp_split_to_array($1, E'\\s+')))), ' ')
 $_$;
 
 
@@ -1326,43 +1326,6 @@ CREATE SEQUENCE public.talent_releases_id_seq
 ALTER SEQUENCE public.talent_releases_id_seq OWNED BY public.talent_releases.id;
 
 
---
--- Name: task_requests; Type: TABLE; Schema: public; Owner: -
---
-
-CREATE TABLE public.task_requests (
-    id bigint NOT NULL,
-    project_id bigint,
-    description text,
-    deadline timestamp without time zone,
-    time_allowed character varying,
-    additional_notes text,
-    status integer DEFAULT 0,
-    created_at timestamp(6) without time zone NOT NULL,
-    updated_at timestamp(6) without time zone NOT NULL,
-    deliverable_url character varying
-);
-
-
---
--- Name: task_requests_id_seq; Type: SEQUENCE; Schema: public; Owner: -
---
-
-CREATE SEQUENCE public.task_requests_id_seq
-    START WITH 1
-    INCREMENT BY 1
-    NO MINVALUE
-    NO MAXVALUE
-    CACHE 1;
-
-
---
--- Name: task_requests_id_seq; Type: SEQUENCE OWNED BY; Schema: public; Owner: -
---
-
-ALTER SEQUENCE public.task_requests_id_seq OWNED BY public.task_requests.id;
-
-
 --
 -- Name: terms; Type: TABLE; Schema: public; Owner: -
 --
@@ -1596,9 +1559,9 @@ CREATE TABLE public.zoom_meetings (
     created_at timestamp(6) without time zone NOT NULL,
     updated_at timestamp(6) without time zone NOT NULL,
     broadcast_id bigint,
-    status integer DEFAULT 0,
     zoom_user_id bigint,
-    project_id bigint
+    project_id bigint,
+    status integer DEFAULT 0
 );
 
 
@@ -1629,7 +1592,9 @@ CREATE TABLE public.zoom_users (
     id bigint NOT NULL,
     api_id character varying,
     created_at timestamp(6) without time zone NOT NULL,
-    updated_at timestamp(6) without time zone NOT NULL
+    updated_at timestamp(6) without time zone NOT NULL,
+    account_number character varying,
+    tier integer DEFAULT 0
 );
 
 
@@ -1869,13 +1834,6 @@ ALTER TABLE ONLY public.tags ALTER COLUMN id SET DEFAULT nextval('public.tags_id
 ALTER TABLE ONLY public.talent_releases ALTER COLUMN id SET DEFAULT nextval('public.talent_releases_id_seq'::regclass);
 
 
---
--- Name: task_requests id; Type: DEFAULT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.task_requests ALTER COLUMN id SET DEFAULT nextval('public.task_requests_id_seq'::regclass);
-
-
 --
 -- Name: terms id; Type: DEFAULT; Schema: public; Owner: -
 --
@@ -2204,14 +2162,6 @@ ALTER TABLE ONLY public.talent_releases
     ADD CONSTRAINT talent_releases_pkey PRIMARY KEY (id);
 
 
---
--- Name: task_requests task_requests_pkey; Type: CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.task_requests
-    ADD CONSTRAINT task_requests_pkey PRIMARY KEY (id);
-
-
 --
 -- Name: terms terms_pkey; Type: CONSTRAINT; Schema: public; Owner: -
 --
@@ -2815,13 +2765,6 @@ CREATE INDEX index_talent_releases_on_term_id ON public.talent_releases USING bt
 CREATE INDEX index_talent_releases_on_territory_id ON public.talent_releases USING btree (territory_id);
 
 
---
--- Name: index_task_requests_on_project_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE INDEX index_task_requests_on_project_id ON public.task_requests USING btree (project_id);
-
-
 --
 -- Name: index_unreleased_appearances_on_video_id; Type: INDEX; Schema: public; Owner: -
 --
@@ -3559,5 +3502,6 @@ INSERT INTO "schema_migrations" (version) VALUES
 ('20200428091105'),
 ('20200507110804'),
 ('20200512161738'),
-('20200518200245'),
+('20200526113516');
-('20200519191908');
+
+

lib/zoom_gateway.rb

@@ -3,6 +3,7 @@ class ZoomGateway
   class MeetingExpired < StandardError; end
   class UserNotFound < StandardError; end
   class TooManyHosts < StandardError; end
+  class UserAlreadyExists < StandardError; end
 
   class << self
     def USER_TYPE_NAME
@@ -23,6 +24,14 @@ class ZoomGateway
       "#{self.USER_TYPE_NAME}-directme-host"
     end
 
+    def ACCOUNT_NUMBER
+      ENV['ZOOM_ACCOUNT_NUMBER']
+    end
+
+    def enable_recordings?
+      ENV['ZOOM_ENABLE_RECORDINGS'] == 'true'
+    end
+
     def apply_limits?
       self.USER_TYPE_NAME == 'pro'
     end
@@ -33,12 +42,14 @@ class ZoomGateway
   end
 
   def create_meeting(host_id, **kwargs)
+    recording_type = self.class.enable_recordings? ? 'cloud' : 'none'
     meeting = @client.meeting_create({ user_id: host_id,
                                        topic: kwargs[:topic],
                                        type: 1, # Instant meeting
                                        settings: {
                                          host_video: true,
-                                         participant_video: true
+                                         participant_video: true,
+                                         auto_recording: recording_type,
                                        } })
     meeting["id"]
   end
@@ -78,6 +89,10 @@ class ZoomGateway
     @client.user_delete(id: host_id)
   end
 
+  def delete_recording(meeting_id, recording_id)
+    @client.recording_delete(meeting_id: meeting_id, recording_id: recording_id)
+  end
+
   private
 
   def parse_zoom_error(error)
@@ -87,6 +102,8 @@ class ZoomGateway
       raise UserNotFound, error.message
     elsif error.status_code == 3001
       raise MeetingExpired, error.message
+    elsif error.status_code == 1005
+      raise UserAlreadyExists, error.message
     else
       raise error
     end

spec/channels/broadcasts_channel_spec.rb

@@ -50,4 +50,21 @@ RSpec.describe BroadcastsChannel, type: :channel do
       })
     end
   end
+
+
+  describe ".broadcast_file_upload_updates" do
+    it 'broadcasts to the channel with the right data' do
+      broadcast = create(:broadcast, :with_stream, :with_files, skip_create_callback: true)
+      files_content = ApplicationController.render partial: "broadcasts/file", collection: broadcast.files
+
+      expect {
+        BroadcastsChannel.broadcast_file_upload_updates(broadcast, broadcast.files, "pagination_content")
+      }.to have_broadcasted_to(broadcast).with({
+        event: 'file_upload_update',
+        broadcast_token: broadcast.token,
+        files_content: files_content,
+        pagination_content: "pagination_content"
+      })
+    end
+  end
 end

spec/channels/projects_channel_spec.rb

@@ -24,4 +24,20 @@ RSpec.describe ProjectsChannel, type: :channel do
       }.to have_broadcasted_to(project).with(event: "video_status_update", content: content)
     end
   end
+
+  describe ".conference_recording_ready" do
+    it "broadcasts to the project channel" do
+      recording_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "video_file.mp4"), "video/mp4")
+      zoom_meeting = create(:zoom_meeting, project: project, recording: recording_file)
+      video_url = 'http://url.to.video/video.mp4'
+      allow(zoom_meeting.recording).to receive(:service_url).and_return(video_url)
+
+      expect {
+        ProjectsChannel.conference_recording_ready(project, zoom_meeting.recording)
+      }.to have_broadcasted_to(project).with { |data|
+        expect(data['content']).to include(video_url)
+        expect(data['event']).to eq('conference_recording_ready')
+      }
+    end
+  end
 end

spec/controllers/admin/task_requests_controller_spec.rb

@@ -1,68 +0,0 @@
-require "rails_helper"
-
-RSpec.describe Admin::TaskRequestsController, type: :controller do
-  render_views
-
-  let!(:current_user) { create(:user, :admin) }
-
-  before do
-    sign_in(current_user)
-  end
-
-  describe "#index" do
-    it "returns a successful response" do
-      get :index
-
-      expect(response).to be_successful
-    end
-  end
-
-  describe "#edit" do
-    let(:task_request) { create(:task_request) }
-
-    it "returns a successful response" do
-      get :edit, params: { id: task_request }
-
-      expect(response).to be_successful
-    end
-
-    it "assigns task request" do
-      get :edit, params: { id: task_request }
-
-      expect(assigns(:task_request)).to eq task_request
-    end
-  end
-
-  describe "#update" do
-    let(:task_request) { create(:task_request) }
-
-    it "redirects to task requests page" do
-      patch :update, params: { id: task_request, task_request: update_params }
-
-      expect(response).to be_redirect
-      expect(response).to redirect_to admin_task_requests_path
-    end
-
-    it "sets a flash notice" do
-      patch :update, params: { id: task_request, task_request: update_params }
-
-      expect(flash.notice).to eq "The task request has been updated successfully"
-    end
-
-    it "updates task request" do
-      patch :update, params: { id: task_request, task_request: update_params }
-
-      expect(task_request.reload.status).to eq("completed")
-      expect(task_request.reload.deliverable_url).to eq("example.com/deliverables")
-    end
-  end
-
-  private
-
-  def update_params
-    {
-      status: "completed",
-      deliverable_url: "example.com/deliverables"
-    }
-  end
-end

spec/controllers/appearance_releases_controller_spec.rb

@@ -26,8 +26,7 @@ RSpec.describe AppearanceReleasesController, tye: :controller do
       get :index, params: { project_id: project }
 
       expect(response.body).to have_content "John Doe"
-      expect(response.body).to have_content "555-123-4567"
+      expect(response.body).to have_content "P: 5551234567E: jane.doe@test.com"
-      expect(response.body).to have_content "jane.doe@test.com"
       expect(response.body).to have_content "Some notes here"
       expect(response.body).to have_button "Import Release"
       expect(response.body).to have_content "Manage"

spec/controllers/broadcasts_controller_spec.rb

@@ -111,17 +111,6 @@ RSpec.describe BroadcastsController, type: :controller do
     end
   end
 
-  describe "#update" do
-    let!(:broadcast) { create(:broadcast, :with_stream, skip_create_callback: true, project: project ) }
-
-    it "uploads files to broadcast" do
-      patch :update, params: { project_id: project.id, id: broadcast.id, broadcast: file_params }, xhr: true
-
-      expect(broadcast.files.count).to eq(1)
-      expect(broadcast.files.first.filename).to eq("contract.pdf")
-    end
-  end
-
   describe "#show" do
     let(:broadcast) { create(:broadcast, project: project, name: "Another Broadcast") }
 
@@ -198,6 +187,25 @@ RSpec.describe BroadcastsController, type: :controller do
     end
   end
 
+  describe "#update" do
+    let!(:broadcast) { create(:broadcast, :with_stream, skip_create_callback: true, project: project ) }
+
+    it "uploads files to broadcast" do
+      patch :update, params: { project_id: project.id, id: broadcast.id, broadcast: file_params }, xhr: true
+
+      expect(broadcast.files.count).to eq(1)
+      expect(broadcast.files.first.filename).to eq("contract.pdf")
+    end
+
+    it "sends an update to the broadcasts channel" do
+      allow(BroadcastsChannel).to receive(:broadcast_file_upload_updates)
+
+      patch :update, params: { project_id: project.id, id: broadcast.id, broadcast: file_params }, xhr: true
+
+      expect(BroadcastsChannel).to have_received(:broadcast_file_upload_updates)
+    end
+  end
+
   describe "#destroy" do
     let!(:broadcast_2) { create(:broadcast, :with_stream, skip_create_callback: true, project: project, name: "Another Broadcast") }
 

spec/controllers/password_resets_controller_spec.rb

@@ -56,6 +56,16 @@ RSpec.describe PasswordResetsController, type: :controller do
       expect(response.body).to have_content("Password Reset")
       expect(response.body).to have_selector("input[name='password_reset[password]']")
     end
+
+    context "when reset token is invalid" do
+      it "redirects to the sign in page" do
+        get :edit, params: { id: "bad token" }
+
+        expect(response).to be_redirect
+        expect(response).to redirect_to(new_session_path)
+        expect(flash.notice).to be_present
+      end
+    end
   end
 
   describe "#update" do

spec/controllers/public/broadcasts_controller_spec.rb

@@ -92,6 +92,14 @@ RSpec.describe Public::BroadcastsController, type: :controller do
       expect(broadcast.files.count).to eq(1)
       expect(broadcast.files.first.filename).to eq("contract.pdf")
     end
+
+    it "sends an update to the broadcasts channel" do
+      allow(BroadcastsChannel).to receive(:broadcast_file_upload_updates)
+
+      patch :update, params: { token: broadcast.token, broadcast: file_params }, xhr: true
+
+      expect(BroadcastsChannel).to have_received(:broadcast_file_upload_updates)
+    end
   end
 
   after do

spec/controllers/task_requests_controller_spec.rb

@@ -1,130 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe TaskRequestsController, 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 "#index" do
-    it "responds successfully" do
-      get :index, params: { project_id: project }
-
-      expect(response).to be_successful
-    end
-
-    it "renders content" do
-      create(:task_request, project: project, description: "Another Request")
-
-      get :index, params: { project_id: project }
-
-      expect(response.body).to have_link "Create Task Request"
-      expect(response.body).to have_content "Pending"
-    end
-
-    context "when there are many records" do
-      it "paginates the table" do
-        create_list(:task_request, 20, project: project)
-
-        get :index, params: { project_id: project }
-
-        expect(response.body).to have_link("2", href: project_task_requests_path(project, page: 2))
-      end
-    end
-  end
-
-  describe "#new" do
-    it "responds successfully" do
-      get :new, params: { project_id: project }
-
-      expect(response).to be_successful
-      expect(assigns(:task_request)).to be_a_new(TaskRequest)
-      expect(response).to render_template(:new)
-    end
-  end
-
-  describe "#create" do
-    it "responds with a redirect" do
-      post :create, params: { project_id: project.id, task_request: task_request_params }
-
-      expect(response).to be_redirect
-      expect(flash.notice).not_to be_nil
-    end
-
-    it "does create a new record" do
-      expect {
-        post :create, params: { project_id: project.id, task_request: task_request_params }
-      }.to change(TaskRequest, :count)
-    end
-
-    it "logs an event" do
-      expect {
-        post :create, params: { project_id: project.id, task_request: task_request_params }
-      }.to have_enqueued_job(TrackAnalyticsJob).with(user,  account, :track_create_task_request, user_agent: "Rails Testing", user_ip: "0.0.0.0")
-    end
-  end
-
-  describe "#update" do
-    let!(:task_request) { create(:task_request, project: project, description: "My description" ) }
-
-    it "updates task request" do
-      patch :update, params: { project_id: project.id, id: task_request.id, task_request: update_params }
-
-      expect(task_request.reload.description).to eq("This is updated description")
-    end
-  end
-
-  describe "#show" do
-    let!(:task_request) { create(:task_request, project: project, description: "Task Request") }
-
-    it "responds successfully" do
-      get :show, params: { project_id: project.id, id: task_request.id }
-
-      expect(response).to be_successful
-      expect(assigns(:task_request)).to eq(task_request)
-    end
-
-    it "renders content" do
-      get :show, params: { project_id: project.id, id: task_request.id }
-      
-      expect(response.body).to have_content "Task Request"
-      expect(response.body).to have_content "pending"
-    end
-  end
-
-  describe "#cancel" do
-    let!(:task_request) { create(:task_request, project: project, description: "Task to be cancelled") }
-
-    it "responds with redirect" do
-      post :cancel, params: { project_id: project.id, id: task_request.id }
-
-      expect(response).to be_redirect
-      expect(response).to redirect_to(project_task_requests_path(project))
-      expect(flash.notice).not_to be_nil
-    end
-
-    it "updates the status to 'cancelled'" do
-      expect {
-        post :cancel, params: { project_id: project.id, id: task_request.id }
-      }.to change { task_request.reload.status }.from("pending").to("cancelled")
-    end
-  end
-
-  private
-
-  def task_request_params
-    attributes = attributes_for(:task_request).except(:status)
-    files = 2.times.map { Rack::Test::UploadedFile.new(file_fixture("location_photo.png"), "image/png") }
-
-    attributes.merge({ files: files })
-  end
-
-  def update_params
-    { description: "This is updated description" }
-  end
-end

spec/controllers/zoom_notifications_controller_spec..rb

@@ -69,6 +69,13 @@ RSpec.describe ZoomNotificationsController, type: :controller do
 
           expect(zoom_meeting.reload).to be_ended
         end
+
+        it 'updates the recording when recording complete notification is received' do
+          expect {
+            post :create, params: recording_complete
+          }.to change { zoom_meeting.recording }
+        end
+
       end
     end
   end

spec/factories/task_requests.rb

@@ -1,14 +0,0 @@
-FactoryBot.define do
-  factory :task_request do
-    association :project
-    deadline { 10.days.from_now }
-    time_allowed "10 days"
-    description "Task request"
-    additional_notes "Additional notes"
-    status 0
-
-    trait :with_files do
-      files { [Rack::Test::UploadedFile.new('spec/fixtures/files/contract.pdf', 'application/pdf')] }
-    end
-  end
-end

spec/factories/zoom_users.rb

@@ -1,5 +1,9 @@
+require 'zoom_gateway'
 FactoryBot.define do
   factory :zoom_user do
+    account_number ZoomGateway.ACCOUNT_NUMBER
+    trait ZoomGateway.USER_TYPE_NAME
+
     trait :with_api_id do
       api_id "api_user_id"
     end

spec/features/admin_managing_task_requests_spec.rb

@@ -1,48 +0,0 @@
-require "rails_helper"
-
-feature "Admin managing task requests" do
-  let(:current_user) { create(:user, admin: true, email: "user@test.com") }
-  let(:project) { create(:project, account: current_user.primary_account, name: "Test Project") }
-
-  before do
-    sign_in current_user
-  end
-
-  scenario "task requests table is visible" do
-    visit admin_task_requests_path
-
-    expect(page).to have_content "Task ID"
-    expect(page).to have_content "Account Name"
-    expect(page).to have_content "Project Name"
-    expect(page).to have_content "Created On"
-    expect(page).to have_content "Deadline"
-    expect(page).to have_content "Time Allowed"
-    expect(page).to have_content "Status"
-  end
-
-  scenario "sees list of task requests" do
-    visit admin_task_requests_path
-
-    expect(page).to have_content no_task_requests_label
-
-    task_request = create(:task_request)
-
-    visit admin_task_requests_path
-
-    expect(page).not_to have_content no_task_requests_label
-
-    expect(page).to have_content task_request.id
-    expect(page).to have_content task_request.project.account.name
-    expect(page).to have_content task_request.project.name
-    expect(page).to have_content task_request.created_at.try(:strftime, '%D')
-    expect(page).to have_content task_request.deadline.try(:strftime, '%D')
-    expect(page).to have_content task_request.time_allowed
-    expect(page).to have_content task_request.status.capitalize
-  end
-
-  private
-
-  def no_task_requests_label
-    "Task requests will appear here"
-  end
-end

spec/features/user_managing_acquired_media_releases_spec.rb

@@ -5,6 +5,13 @@ feature "User managing acquired_media releases" do
   let(:project) { create(:project, members: current_user, account: current_user.primary_account) }
 
   context "when signed out" do
+    scenario "United States is default country" do
+      contract_template = create(:contract_template, project: project)
+
+      visit new_account_project_contract_template_acquired_media_release_path(project.account, project, contract_template)
+      expect(country_field_value).to eq "US"
+    end
+
     scenario "creating a release", js: true do
       contract_template = create(:contract_template, project: project)
 
@@ -19,7 +26,7 @@ feature "User managing acquired_media releases" do
 
       click_button "I have read and agree to the above"
 
-      expect(AcquiredMediaRelease.last.categories).to include("Artwork")
+      expect(AcquiredMediaRelease.last.categories).to include("Video Footage")
       expect(AcquiredMediaRelease.last.categories).to include("Still Photograph")
       expect(page).to have_content("Your release was successfully submitted. Thank you.")
     end
@@ -205,12 +212,16 @@ feature "User managing acquired_media releases" do
 
   private
 
+  def country_field_value
+    find_field("acquired_media_release[person_address_country]").value
+  end
+
   def acquired_media_name_field
     "acquired_media_release[name]"
   end
 
   def acquired_media_category_fields
-    find(:css, "#acquired_media_release_categories_artwork").set(true)
+    find(:css, "#acquired_media_release_categories_video_footage").set(true)
     find(:css, "#acquired_media_release_categories_still_photograph").set(true)
   end
 

spec/features/user_managing_broadcasts_spec.rb

@@ -46,15 +46,17 @@ feature "User managing broadcasts" do
       expect(page).to have_content("Live stream is waiting to begin.")
       expect(page).to have_content("Copy URL")
       
-      click_on "Files"
+      within "#files" do
-      expect(page).to have_content("contract.pdf")
+        expect(page).to have_content("contract.pdf")
+      end
 
       click_on "Previous Sessions"
       expect(page).to have_content(recording.download_file_name)
     end
 
     scenario "visit multi-view broadcast page", js: true do
-      broadcasts = create_list(:broadcast, 4, :with_stream, project: project)
+      broadcast_one = create(:broadcast, :with_stream, :with_files, name: "Broadcast 1", project: project)
+      broadcast_two = create(:broadcast, :with_stream, :with_files, name: "Broadcast 2", project: project)
       
       visit project_broadcasts_path(project)
       click_checkboxes
@@ -64,8 +66,16 @@ feature "User managing broadcasts" do
         expect(page).to have_content("Switch View")
 
         click_on "Switch View"
-        expect(page).to have_link(broadcasts.first.name)
+        expect(page).to have_link("Broadcast 1")
-        expect(page).to have_link(broadcasts.second.name)
+        expect(page).to have_link("Broadcast 2")
+
+        within "#files" do
+          click_on "Broadcast 1"
+          expect(page).to have_content("contract.pdf")
+
+          click_on "Broadcast 2"
+          expect(page).to have_content("contract.pdf")
+        end
       end 
     end
   end
@@ -77,7 +87,7 @@ feature "User managing broadcasts" do
   end
 
   def click_checkboxes
+    all('input[type="checkbox"]')[0].click
     all('input[type="checkbox"]')[1].click
-    all('input[type="checkbox"]')[2].click
   end
 end

spec/features/user_managing_location_releases_spec.rb

@@ -5,6 +5,13 @@ feature "User managing location releases" do
   let(:project) { create(:project, members: current_user, account: current_user.primary_account) }
 
   context "when signed out" do
+    scenario "United States is default country" do
+      contract_template = create(:contract_template, project: project)
+
+      visit new_account_project_contract_template_location_release_path(project.account, project, contract_template)
+      expect(country_field_value).to eq "US"
+    end
+
     scenario "creating a release", js: true do
       contract_template = create(:contract_template, project: project)
 
@@ -17,6 +24,7 @@ feature "User managing location releases" do
         fill_in person_phone_field, with: "555-555-5555"
         fill_in person_email_field, with: "jane.doe@test.com"
         fill_in person_address_street1_field, with: "100 Broadway"
+        fill_in filming_hours_field, with: "04:00 - 22:00"
         draw_signature file_fixture("signature.png"), "location_release_signature_base64"
       end
 
@@ -51,6 +59,7 @@ feature "User managing location releases" do
 
       by "filling out the remaining information" do
           fill_in_release_fields name: "Test Location Release"
+          fill_in filming_hours_field, with: "04:00 - 22:00"
           click_button create_release_button
           expect(page).to have_content(create_release_notice)
           expect(page).to have_photo("location_photo.png")
@@ -136,6 +145,7 @@ feature "User managing location releases" do
                               :native,
                               project: project,
                               name: "Benny's Burritos",
+                              filming_hours: "06:00 - 20:00",
                               tag_list: "Restaurant",
                               notes: [
                                 build(:note, 
@@ -172,6 +182,8 @@ feature "User managing location releases" do
     expect(pdf_body).to have_content("Restaurant")
     expect(pdf_body).to have_content photos_heading.upcase
     expect(pdf_body).to have_content("location_photo.png")
+    expect(pdf_body).to have_content("Filming Hours")
+    expect(pdf_body).to have_content("06:00 - 20:00")
   end
 
   context "when the user is associate" do
@@ -190,6 +202,10 @@ feature "User managing location releases" do
 
   private
 
+  def country_field_value
+    find_field("location_release[person_address_country]").value
+  end
+
   def photos_heading(photos_count = 1)
     t 'contracts.photos.heading', count: photos_count
   end
@@ -222,6 +238,10 @@ feature "User managing location releases" do
     "location_release[person_phone]"
   end
 
+  def filming_hours_field
+    "location_release[filming_hours]"
+  end
+
   def have_photo(filename, attr: "src")
     have_selector("img[#{attr}*='#{filename}']")
   end

spec/features/user_managing_material_releases_spec.rb

@@ -5,6 +5,13 @@ feature "User managing material releases" do
   let(:project) { create(:project, members: current_user, account: current_user.primary_account) }
 
   context "when signed out" do
+    scenario "United States is default country" do
+      contract_template = create(:contract_template, project: project)
+
+      visit new_account_project_contract_template_material_release_path(project.account, project, contract_template)
+      expect(country_field_value).to eq "US"
+    end
+
     scenario "creating a release", js: true do
       contract_template = create(:contract_template, project: project)
 
@@ -180,6 +187,10 @@ feature "User managing material releases" do
 
   private
 
+  def country_field_value
+    find_field("material_release[person_address_country]").value
+  end
+
   def photos_heading(photos_count = 1)
     t 'contracts.photos.heading', count: photos_count
   end

spec/features/user_managing_talent_releases_spec.rb

@@ -5,6 +5,13 @@ feature "User managing talent releases" do
   let(:project) { create(:project, members: current_user, account: current_user.primary_account) }
 
   context "when signed out" do
+    scenario "United States is default country" do
+      contract_template = create(:contract_template, project: project)
+
+      visit new_account_project_contract_template_talent_release_path(project.account, project, contract_template)
+      expect(country_field_value).to eq "US"
+    end
+
     scenario "creating a release for an adult", js: true do
       contract_template = create(:contract_template, project: project)
 
@@ -274,6 +281,10 @@ feature "User managing talent releases" do
 
   private
 
+  def country_field_value
+    find_field("talent_release[person_address_country]").value
+  end
+
   def photos_heading(photos_count = 1)
     t 'contracts.photos.heading', count: photos_count
   end

spec/features/user_performs_video_analysis_spec.rb

@@ -521,8 +521,8 @@ feature "User performs video analysis" do
       new_path = polymorphic_path([:new, video, release, :video_release_confirmation])
       first("form[action='#{new_path}']").click_button
 
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
 
       click_on "Show/Hide EDL Events"
       within "#edl_events" do
@@ -572,8 +572,8 @@ feature "User performs video analysis" do
       new_path = polymorphic_path([:new, video, release, :video_release_confirmation])
       first("form[action='#{new_path}']").click_button
 
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
 
       click_on "Show/Hide EDL Events"
       within "#edl_events" do
@@ -748,8 +748,8 @@ feature "User performs video analysis" do
 
       open_graphics_element_modal(video)
 
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
 
       click_on "Show/Hide EDL Events"
       within "#edl_events" do
@@ -796,8 +796,8 @@ feature "User performs video analysis" do
 
       open_graphics_element_modal(video)
 
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
 
       click_on "Show/Hide EDL Events"
       within "#edl_events" do
@@ -1188,8 +1188,8 @@ feature "User performs video analysis" do
       open_audio_confirmation_modal
 
       expect(page).to have_field("Origin", with: "Original Music")
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
       expect(page).to have_field("Source EDL", with: "Audio")
 
       click_on "Show/Hide EDL Events"
@@ -1219,10 +1219,10 @@ feature "User performs video analysis" do
         expect(page).to have_field("Source EDL", with: "All Tracks")
         expect(page).to have_field("Channel", with: "A1")
         expect(page).to have_field("Origin", with: "Library Music")
-        expect(page).to have_field("Timecode in", with: "01:00:00:00")
+        expect(page).to have_field("Timecode In", with: "01:00:00:00")
         expect(page).to have_field("Timecode out", with: "01:00:02:00")
         expect(page).to have_field("Duration", with: "00:00:02")
-        expect(page).to have_field("Source file name", with: "source_file_name.wav")
+        expect(page).to have_field("Source File Name", with: "source_file_name.wav")
         expect(page).to have_field("Clip name", with: "clip_name")
         expect(page).to have_field("Description", with: "description")
 
@@ -1323,8 +1323,8 @@ feature "User performs video analysis" do
 
       open_unreleased_modal(video)
 
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
 
       click_on "Show/Hide EDL Events"
       within "#edl_events" do
@@ -1371,8 +1371,8 @@ feature "User performs video analysis" do
 
       open_unreleased_modal(video)
 
-      expect(page).to have_content "Timecode in"
+      expect(page).to have_content "Timecode In"
-      expect(page).to have_content "Source file name"
+      expect(page).to have_content "Source File Name"
 
       click_on "Show/Hide EDL Events"
       within "#edl_events" do
@@ -1445,7 +1445,7 @@ feature "User performs video analysis" do
 
     first("#suggested_matches form[action='#{new_path}']").click_button
 
-    expect(page).to have_content "Timecode in"
+    expect(page).to have_content "Timecode In"
 
     within "form[action='#{create_path}']" do
       click_button "Confirm Release"
@@ -1462,7 +1462,7 @@ feature "User performs video analysis" do
       $("#audio_matches .releasable-match button")[0].click()
     JS
 
-    expect(page).to have_content "Timecode in"
+    expect(page).to have_content "Timecode In"
   end
 
   def select_option(select_id, text)
@@ -1486,7 +1486,7 @@ feature "User performs video analysis" do
     JS
 
     expect(page).to have_field("Origin", with: "Library Music")
-    expect(page).to have_content "Timecode in"
+    expect(page).to have_content "Timecode In"
 
     within "form[action='#{create_path}']" do
       expect(page).to have_selector("#matched_file_name", text: "library_file_name")
@@ -1518,7 +1518,7 @@ feature "User performs video analysis" do
 
     first("form[action='#{new_path}']", text: text).click_button
 
-    expect(page).to have_content "Timecode in"
+    expect(page).to have_content "Timecode In"
 
     within "form[action='#{create_path}']" do
       click_button "Confirm Release"
@@ -1536,7 +1536,7 @@ feature "User performs video analysis" do
     first("#music_releases form[action='#{new_path}']", text: text).click_button
 
     expect(page).to have_field("Origin", with: "Original Music")
-    expect(page).to have_content "Timecode in"
+    expect(page).to have_content "Timecode In"
 
     within "form[action='#{create_path}']" do
       select "Vocal", from: "Music type"

spec/jobs/attach_recording_to_zoom_meeting_job_spec.rb

@@ -0,0 +1,45 @@
+require "rails_helper"
+
+describe AttachRecordingToZoomMeetingJob do
+  let(:project) { create(:project) }
+  let(:zoom_meeting) { create(:zoom_meeting, project: project) }
+  let(:recording_hash) { {'id' => 'recording-id', 'download_url' => 'http://download.url', 'recording_start' => '2020-05-22 16:33:50 UTC', 'recording_end' => '2020-05-22 17:34:06 UTC'} }
+  let(:download_token) { 'download_token' }
+
+  before :each do
+    allow_any_instance_of(ZoomGateway).to receive(:delete_recording)
+  end
+
+  describe ".perform_now" do
+    it "downloads the video" do
+      allow(zoom_meeting.recording).to receive(:attach)
+
+      expect(URI).to receive(:open).with("http://download.url?access_token=download_token")
+      AttachRecordingToZoomMeetingJob.perform_now zoom_meeting, recording_hash, download_token
+    end
+
+    it "deletes the recording through the API" do
+      stub_uri_open
+      allow(zoom_meeting.recording).to receive(:attach).and_return(true)
+
+      expect_any_instance_of(ZoomGateway).to receive(:delete_recording).with(zoom_meeting.api_meeting_id, 'recording-id')
+      AttachRecordingToZoomMeetingJob.perform_now zoom_meeting, recording_hash, download_token
+    end
+
+    it "attaches downloaded video to recording" do
+      allow(zoom_meeting.recording).to receive(:attach)
+      stub_uri_open
+
+      AttachRecordingToZoomMeetingJob.perform_now zoom_meeting, recording_hash, download_token
+      expect(zoom_meeting.recording).to have_received(:attach).with(hash_including(content_type: 'video/mp4'))
+    end
+  end
+
+  private
+
+  def stub_uri_open
+    url = "http://download.url?access_token=download_token"
+    file = double(:file, read: 'stubbed read')
+    allow(URI).to receive(:open).with(url).and_return(file)
+  end
+end

spec/lib/zoom_gateway_spec.rb

@@ -64,6 +64,28 @@ RSpec.describe ZoomGateway do
         expect(ZoomGateway.HOST_ROLE).to eq('pro-directme-host')
       end
     end
+
+    context '.enable_recordings?' do
+      it 'is truthy when ZOOM_ENABLE_RECORDINGS is set to true' do
+        stub_env_variable('ZOOM_ENABLE_RECORDINGS', 'true')
+        expect(ZoomGateway.enable_recordings?).to be_truthy
+      end
+
+      it 'is falsey otherwise' do
+        stub_env_variable('ZOOM_ENABLE_RECORDINGS', '1')
+        expect(ZoomGateway.enable_recordings?).to be_falsey
+
+        stub_env_variable('ZOOM_ENABLE_RECORDINGS', 'false')
+        expect(ZoomGateway.enable_recordings?).to be_falsey
+      end
+    end
+
+    context '.ACCOUNT_NUMBER' do
+      it 'depends on ENV["ZOOM_ACCOUNT_NUMBER"]' do
+        stub_env_variable('ZOOM_ACCOUNT_NUMBER', 'xxx-yyy-zzz')
+        expect(ZoomGateway.ACCOUNT_NUMBER).to eq('xxx-yyy-zzz')
+      end
+    end
   end
 
   describe ".find_meeting" do
@@ -102,20 +124,18 @@ RSpec.describe ZoomGateway do
       end
 
       it 'raises an exception' do
-        allow(ENV).to receive(:[]).and_call_original
+        stub_env_variables(ZOOM_USER_TYPE: 'pro', ZOOM_PRO_USERS_LIMIT: 2)
-        allow(ENV).to receive(:[]).with('ZOOM_USER_TYPE').and_return('pro')
-        allow(ENV).to receive(:[]).with('ZOOM_PRO_USERS_LIMIT').and_return('2')
 
         expect { gateway.create_host('host-email@address') }.to raise_error(ZoomGateway::TooManyHosts)
       end
     end
   end
 
-  private
+  describe '.delete_recording' do
-
+    it 'calls api client to delete recording' do
-  def stub_env_variable(name, value)
+      expect_any_instance_of(Zoom.new.class).to receive(:recording_delete)
-    allow(ENV).to receive(:[]).and_call_original
+      gateway.delete_recording('meeting-id', 'recording-id')
-    allow(ENV).to receive(:[]).with(name.to_s).and_return(value.to_s)
+    end
   end
 
 end

spec/models/account_spec.rb

@@ -66,14 +66,16 @@ RSpec.describe Account do
   end
 
   describe "#storage_total" do
-    it "sums videos, release photos, contracts, signatures" do
+    it "sums videos, release photos, contracts, signatures, recordings" do
       video_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "video_file.mp4"), "video/mp4")
+      recording_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "video_file.mp4"), "video/mp4")
       photo_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "person_photo.png"), "image/png")
       contract_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "contract.pdf"), "application/pdf")
       signature_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "signature.png"), "image/png")
       edl_file = Rack::Test::UploadedFile.new(Rails.root.join("spec", "fixtures", "files", "sample-edl.edl"), "application/octet-stream")
 
       expect(video_file.size).to eq 1_055_736
+      expect(recording_file.size).to eq 1_055_736
       expect(edl_file.size).to eq 440
       expect(photo_file.size).to eq 910
       expect(contract_file.size).to eq 12
@@ -89,8 +91,9 @@ RSpec.describe Account do
       acquired_media_release = create(:acquired_media_release, project: project, contract: contract_file)
       import = create(:import, project: project, file: contract_file)
       music_release = create(:music_release, project: project, contract: contract_file)
+      zoom_meeting = create(:zoom_meeting, project: project, recording: recording_file)
 
-      expect(account.storage_total).to eq 1_069_936
+      expect(account.storage_total).to eq 2_125_672
     end
 
     it "sums only for projects tied to account" do
@@ -126,8 +129,8 @@ RSpec.describe Account do
         Download,
         User,
         Broadcast,
-        TaskRequest,
+        Account,
-        Account
+        ZoomMeeting
       ]
       Rails.application.eager_load!
       ActiveRecord::Base.descendants.each do |model|

spec/models/app_host_spec.rb

@@ -147,10 +147,4 @@ describe AppHost do
       expect(app_domain.protocol).to eq :http
     end
   end
-
-  private
-
-  def stub_env(key, value)
-    allow(ENV).to receive(:fetch).with(key).and_return(value)
-  end
 end

spec/models/headshot_collection_spec.rb

@@ -6,6 +6,7 @@ describe HeadshotCollection do
       project = create(:project, 
                        appearance_releases: create_list(:appearance_release, 1),
                        talent_releases: create_list(:talent_release, 1),
+                       headshot_collection_uid: "123abc"
                       )
 
       collection = HeadshotCollection.for_project(project)
@@ -14,7 +15,7 @@ describe HeadshotCollection do
       expect(collection).to be_a(HeadshotCollection)
       expect(collection.releasables).to include(project.appearance_releases.first)
       expect(collection.releasables).to include(project.talent_releases.first)
-      expect(collection.collection_uid).to eq(project.id)
+      expect(collection.collection_uid).to eq(project.headshot_collection_uid)
     end
 
     context "when a release has no headshot photo attachment" do
@@ -85,6 +86,17 @@ describe HeadshotCollection do
       expect(mapping["appearance_release_#{releases.first.id}"]).to include("123")
       expect(mapping["talent_release_#{releases.last.id}"]).to include("456")
     end
+
+    context "when collection uid is blank" do
+      it "is not included in the hash" do
+        releases = []
+        collection = HeadshotCollection.new(nil, releases)
+
+        hash = collection.to_hash
+
+        expect(hash.keys).not_to include(:collection_uid)
+      end
+    end
   end
 
   private

spec/models/project_spec.rb

@@ -79,7 +79,7 @@ RSpec.describe Project, type: :model do
 
     context 'there is no meeting' do
       context 'there is a free user available' do
-        let!(:free_zoom_user) { create(:zoom_user, api_id: 'user_id') }
+        let!(:free_zoom_user) { create(:zoom_user, :with_api_id) }
 
         before do
           allow_any_instance_of(ZoomGateway).to receive(:create_meeting).and_return('new-meeting-id')

spec/models/task_request_spec.rb

@@ -1,16 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe TaskRequest, type: :model do
-  describe "associations" do
-    it { is_expected.to belong_to(:project) }
-  end
-
-  describe "enums" do
-    it { is_expected.to define_enum_for(:status).with_values([:pending, :completed, :cancelled]) }
-  end
-
-  describe "#order_by_recent" do
-    subject { described_class }
-    it { is_expected.to respond_to(:order_by_recent) }
-  end
-end

spec/models/zoom_meeting_spec.rb

@@ -16,6 +16,17 @@ RSpec.describe ZoomMeeting, type: :model do
   end
 
 
+  describe "attachments" do
+    it { is_expected.to respond_to(:recording) }
+  end
+
+  describe "validations" do
+    context '#recording' do
+      it { is_expected.to allow_content_type("video/mp4").for(:recording) }
+      it { is_expected.not_to allow_content_types("image/png").for(:recording) }
+    end
+  end
+
   describe 'associations' do
     it { is_expected.to belong_to(:zoom_user) }
     it { is_expected.to belong_to(:project).optional(true) }

spec/models/zoom_user_spec.rb

@@ -1,3 +1,4 @@
+require 'zoom_gateway'
 require 'rails_helper'
 
 RSpec.describe ZoomUser, type: :model do
@@ -5,6 +6,10 @@ RSpec.describe ZoomUser, type: :model do
     it { is_expected.to have_many(:zoom_meetings).dependent(:nullify) }
   end
 
+  describe "enums" do
+    it { is_expected.to define_enum_for(:tier).with_values([:basic, :pro]) }
+  end
+
   describe 'callbacks' do
     let(:zoom_user) { build(:zoom_user) }
 
@@ -22,10 +27,120 @@ RSpec.describe ZoomUser, type: :model do
         zoom_user.save
         expect(zoom_user.api_id).to eq "retrieved_api_id"
       end
+
+      it 'assigns current account number' do
+        allow_any_instance_of(ZoomGateway).to receive(:create_host).and_return("retrieved_api_id")
+        ENV['ZOOM_ACCOUNT_NUMBER'] = 'xxx-yyy-zzz'
+
+        zoom_user.save
+        expect(zoom_user.account_number).to eq 'xxx-yyy-zzz'
+      end
     end
 
     context '#before_destroy' do
       pending 'aborts if there is api_id assigned'
     end
   end
+
+  describe 'scopes' do
+    context '.current_account' do
+      before do
+        create_list(:zoom_user, 10, :with_api_id, account_number: 'first-account-id')
+        create_list(:zoom_user, 25, :with_api_id, account_number: 'second-account-id')
+      end
+
+      it 'only returns users from currently set account' do
+        ENV['ZOOM_ACCOUNT_NUMBER'] = 'first-account-id'
+        expect(ZoomUser.current_account.count).to eq 10
+        expect(ZoomUser.current_account.pluck(:account_number).uniq.count).to eq 1
+        expect(ZoomUser.current_account.pluck(:account_number).uniq.first).to eq 'first-account-id'
+      end
+    end
+
+    context '.free' do
+      let(:free_user) { create(:zoom_user, :with_api_id) }
+      let(:busy_user) { create(:zoom_user, :with_api_id) }
+      let(:second_busy_user) { create(:zoom_user, :with_api_id) }
+      let!(:meeting_started) { create(:zoom_meeting, zoom_user: busy_user, status: :started) }
+      let!(:meeting_created) { create(:zoom_meeting, zoom_user: second_busy_user, status: :created) }
+
+      it 'returns only the users without started / created meetings' do
+        users = ZoomUser.free
+
+        expect(users).to include(free_user)
+        expect(users).not_to include(busy_user)
+        expect(users).not_to include(second_busy_user)
+      end
+    end
+  end
+
+
+  describe '.current_account?' do
+    let(:zoom_user) { create(:zoom_user, :with_api_id, account_number: 'xxx-xxx-xxx') }
+
+    it 'returns true if it belongs to currently set account' do
+      ENV['ZOOM_ACCOUNT_NUMBER'] = 'xxx-xxx-xxx'
+      expect(zoom_user.current_account?).to be_truthy
+    end
+
+    it 'returns false if it doesn\'t belong to currently set account' do
+      ENV['ZOOM_ACCOUNT_NUMBER'] = 'yyy-yyy-yyy'
+      expect(zoom_user.current_account?).to be_falsey
+    end
+  end
+
+
+  context 'static methods' do
+    describe '.generate_api_email' do
+      it 'contains @' do
+        expect(ZoomUser.generate_api_email).to include('@')
+      end
+    end
+
+    describe '.for_new_meeting' do
+      let(:a_basic) { create(:zoom_user, :with_api_id, account_number: 'aaa', tier: :basic) }
+      let(:b_basic) { create(:zoom_user, :with_api_id, account_number: 'bbb', tier: :basic) }
+      let(:a_pro) { create(:zoom_user, :with_api_id, account_number: 'aaa', tier: :pro) }
+      let(:b_pro) { create(:zoom_user, :with_api_id, account_number: 'bbb', tier: :pro) }
+
+      it 'picks free user of requested type from current account' do
+        [a_basic, a_pro, b_basic, b_pro]
+
+        stub_env_variables(ZOOM_USER_TYPE: 'basic', ZOOM_ACCOUNT_NUMBER: 'aaa')
+        expect(ZoomUser.for_new_meeting).to eq(a_basic)
+
+        stub_env_variables(ZOOM_USER_TYPE: 'pro', ZOOM_ACCOUNT_NUMBER: 'aaa')
+        expect(ZoomUser.for_new_meeting).to eq(a_pro)
+
+        stub_env_variables(ZOOM_USER_TYPE: 'basic', ZOOM_ACCOUNT_NUMBER: 'bbb')
+        expect(ZoomUser.for_new_meeting).to eq(b_basic)
+
+        stub_env_variables(ZOOM_USER_TYPE: 'pro', ZOOM_ACCOUNT_NUMBER: 'bbb')
+        expect(ZoomUser.for_new_meeting).to eq(b_pro)
+      end
+
+      context 'no free user' do
+        before do
+          allow_any_instance_of(ZoomGateway).to receive(:create_host).and_return('host-id')
+        end
+
+        it 'creates new user if there is no requested user free available under account' do
+          [a_basic, a_pro, b_basic]
+          stub_env_variables(ZOOM_USER_TYPE: 'pro', ZOOM_ACCOUNT_NUMBER: 'bbb')
+
+          expect_any_instance_of(ZoomGateway).to receive(:create_host)
+          ZoomUser.for_new_meeting
+        end
+
+        it 'creates new user if requested user is busy' do
+          [a_basic, a_pro, b_basic, b_pro]
+          stub_env_variables(ZOOM_USER_TYPE: 'pro', ZOOM_ACCOUNT_NUMBER: 'aaa')
+          create(:zoom_meeting, zoom_user: a_pro, status: :started)
+
+          expect_any_instance_of(ZoomGateway).to receive(:create_host)
+          ZoomUser.for_new_meeting
+        end
+      end
+    end
+  end
 end

spec/policies/contract_template_policy_spec.rb

@@ -15,6 +15,10 @@ describe ContractTemplatePolicy do
       it { is_expected.not_to permit(user_context, :create) }
     end
 
+    permissions :show? do
+      it { is_expected.to permit(user_context, :show) }
+    end
+
     permissions :destroy? do
       it { is_expected.not_to permit(user_context, contract_template) }
 
@@ -33,6 +37,32 @@ describe ContractTemplatePolicy do
       it { is_expected.to permit(user_context, :create) }
     end
 
+    permissions :show? do
+      it { is_expected.to permit(user_context, :show) }
+    end
+
+    permissions :destroy? do
+      it { is_expected.to permit(user_context, contract_template) }
+
+      context "when there are associated releases" do
+        let(:contract_template) { create(:contract_template, appearance_releases: build_list(:appearance_release, 1)) }
+
+        it { is_expected.to permit(user_context, contract_template) }
+      end
+    end
+  end
+
+  context "for an account manager" do
+    let(:user) { create(:user, :account_manager) }
+
+    permissions :create? do
+      it { is_expected.to permit(user_context, :create) }
+    end
+
+    permissions :show? do
+      it { is_expected.to permit(user_context, :show) }
+    end
+
     permissions :destroy? do
       it { is_expected.to permit(user_context, contract_template) }
 
@@ -79,7 +109,7 @@ describe ContractTemplatePolicy do
     context "for associate" do
       let(:user) { create(:user, :associate, primary_account: account) }
 
-      it { is_expected.not_to include(member_project.contract_templates.first) }
+      it { is_expected.to include(member_project.contract_templates.first) }
       it { is_expected.not_to include(non_member_project.contract_templates.first) }
       it { is_expected.not_to include(outside_project.contract_templates.first) }
     end