effective_cpd 1.0.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b9ef412f128415399ade394ba5ee59d6799d9f13a600904bccfc6c88aa80fd6
-  data.tar.gz: 3c193fe0263c126908e9e2029ea303cfc9842b41d80de7e0bd29761afd97d870
+  metadata.gz: deb0da2dfd0958689944cea459a4e237ac5f765cecb2b5fc081e513b1b648f38
+  data.tar.gz: 913951e4a77ffa9fa00c22fb8c153702c7b05d3a9aefdeb4eaa90d3af0495a7d
 SHA512:
-  metadata.gz: b8f785d86dedc5821fb7118a7e8ea2e7c5baedc29abcd470e0c420c8e1352e51aaa4bcd364a0822e75d013778fcfdd21ad20ab5337ee55223f12cda03bf7bcee
-  data.tar.gz: 8a817518db9ab7b7a7104f3d6eb4991e9422bce880a4d5817bf9dd6e8917a6fd4070f96fdae059bd978bb6b6f6888faf781e0ee8c84ea959fb39ed9be6d0c129
+  metadata.gz: '08548deb0cbcbb5ad26f4f7d962b06ebda9d63e1e230ec7950c212f39ee83b97805bd133e1a9a807e77da7b8173e0e7443c60b3a19c86a93bb1bfa19c411c7ca'
+  data.tar.gz: 9180db8bd3c9cb84fbf13bc136cda24a4996822ff8217b0a3ddfd88501e646160209698ecde6734f32ab2a831aae1e11ec5cbacea96bdc43c973750c983f2c44

data/README.md

@@ -186,8 +186,8 @@ if user.persisted?
 end
 
 if user.reviewer?
-  can [:index], Effective::CpdAudit
-  can [:index, :show, :update], Effective::CpdAuditReview, user_id: user.id
+  can [:index], EffectiveCpd.CpdAudit
+  can [:index, :show, :update], EffectiveCpd.CpdAuditReview, user_id: user.id
 end
 
 if user.admin?
@@ -197,15 +197,21 @@ if user.admin?
   can :manage, Effective::CpdCycle
   can :manage, Effective::CpdRule
 
-  can([:index, :show], Effective::CpdStatement)
-  can(:unsubmit, Effective::CpdStatement) { |statement| statement.completed? }
+  can([:index, :show], EffectiveCpd.CpdStatement)
+  can(:unsubmit, EffectiveCpd.CpdStatement) { |statement| statement.completed? }
 
   can :manage, Effective::CpdAuditLevel
   can :manage, Effective::CpdAuditLevelSection
   can :manage, Effective::CpdAuditLevelQuestion
 
-  can :manage, Effective::CpdAudit
-  can :manage, Effective::CpdAuditReview
+  can(crud - [:destroy], EffectiveCpd.CpdAudit)
+  can(:destroy, EffectiveCpd.CpdAudit) { |audit| audit.draft? }
+
+  can(:missing, EffectiveCpd.CpdAudit) { |audit| audit.was_submitted? && !audit.missing_info? }
+  can(:complete, EffectiveCpd.CpdAudit) { |audit| audit.was_submitted? && !audit.was_completed? }
+  can(:close, EffectiveCpd.CpdAudit) { |audit| audit.was_submitted? && !audit.closed? }
+
+  can(crud - [:edit, :update], EffectiveCpd.CpdAuditReview)
 end
 ```
 

data/app/controllers/admin/cpd_audit_levels_controller.rb

@@ -5,8 +5,8 @@ module Admin
 
     include Effective::CrudController
 
-    resource_scope -> { EffectiveCpd.AuditLevel.deep.all }
-    datatable -> { Admin::EffectiveCpdAuditLevelsDatatable.new }
+    resource_scope -> { EffectiveCpd.CpdAuditLevel.deep.all }
+    datatable -> { EffectiveResources.best('Admin::EffectiveCpdAuditLevelsDatatable').new }
 
     private
 

data/app/controllers/admin/cpd_audit_reviews_controller.rb

@@ -5,8 +5,8 @@ module Admin
 
     include Effective::CrudController
 
-    resource_scope -> { EffectiveCpd.AuditReview.deep.all }
-    datatable -> { Admin::EffectiveCpdAuditReviewsDatatable.new }
+    resource_scope -> { EffectiveCpd.CpdAuditReview.deep.all }
+    datatable -> { EffectiveResources.best('Admin::EffectiveCpdAuditReviewsDatatable').new }
 
     private
 

data/app/controllers/admin/cpd_audits_controller.rb

@@ -6,7 +6,7 @@ module Admin
     include Effective::CrudController
 
     resource_scope -> { EffectiveCpd.CpdAudit.deep.all }
-    datatable -> { Admin::EffectiveCpdAuditsDatatable.new }
+    datatable -> { EffectiveResources.best('Admin::EffectiveCpdAuditsDatatable').new }
 
     submit :resolve_conflict, 'Resolve Conflict of Interest', success: -> {
       [
@@ -36,6 +36,20 @@ module Admin
       ].compact.join(' ')
     }
 
+    submit :complete, 'Complete Audit', success: -> {
+      [
+        "Successfully completed #{resource}",
+        ("and sent reviewers an audit ready notification" unless resource.email_form_skip)
+      ].compact.join(' ')
+    }
+
+    submit :missing, 'Missing Info', success: -> {
+      [
+        "Successfully marked #{resource} as missing info",
+        ("and sent #{resource.user.email} a notification" unless resource.email_form_skip)
+      ].compact.join(' ')
+    }
+
     private
 
     def permitted_params

data/app/controllers/admin/cpd_statements_controller.rb

@@ -6,7 +6,7 @@ module Admin
     include Effective::CrudController
 
     resource_scope -> { EffectiveCpd.CpdStatement.deep.all }
-    datatable -> { Admin::EffectiveCpdStatementsDatatable.new }
+    datatable -> { EffectiveResources.best('Admin::EffectiveCpdStatementsDatatable').new }
 
     page_title(only: :index) { "#{EffectiveCpd.program_label} Statements" }
 

data/app/controllers/effective/cpd_audit_reviews_controller.rb

@@ -5,56 +5,22 @@ module Effective
     include Effective::WizardController
 
     resource_scope do
-      Effective::CpdAuditReview.deep.where(user: current_user)
+      EffectiveCpd.CpdAuditReview.deep.where(user: current_user)
     end
 
+    page_title 'Chat', only: [:chat]
+
     # Reuse the same view for all cpd_audit_level_section steps
     # https://github.com/zombocom/wicked/blob/v1.3.4/lib/wicked/controller/concerns/render_redirect.rb#L32
     def render_step(the_step, options = {}, params = {})
       if resource.dynamic_wizard_statement_steps.keys.include?(the_step)
-        render('cpd_statement', options)
+        render('effective/cpd_audit_reviews/cpd_statement', options)
       elsif resource.dynamic_wizard_questionnaire_steps.keys.include?(the_step)
-        render('cpd_audit_level_section', options)
+        render('effective/cpd_audit_reviews/cpd_audit_level_section', options)
       else
         super
       end
     end
 
-    def permitted_params
-      case step
-      when :start
-        params.require(:effective_cpd_audit_review).permit(:current_step, :comments)
-      when :information
-        params.require(:effective_cpd_audit_review).permit(:current_step, :comments)
-      when :instructions
-        params.require(:effective_cpd_audit_review).permit(:current_step, :comments)
-      when :conflict
-        params.require(:effective_cpd_audit_review)
-          .permit(:current_step, :comments, :conflict_of_interest, :conflict_of_interest_reason)
-      when :statements
-        params.require(:effective_cpd_audit_review).permit(:current_step, :comments)
-      when :questionnaire
-        params.require(:effective_cpd_audit_review).permit(:current_step, :comments)
-      when :recommendation
-        params.require(:effective_cpd_audit_review).permit(:current_step, :comments, :recommendation)
-      when :submit
-        params.require(:effective_cpd_audit_review).permit(:current_step)
-      when :complete
-        raise('unexpected post to complete')
-      else
-        if step.to_s.start_with?('statement')
-          params.require(:effective_cpd_audit_review).permit(:current_step, :comments,
-            cpd_audit_review_items_attributes: [:id, :item_id, :item_type, :recommendation, :comments]
-          )
-        elsif step.to_s.start_with?('section')
-          params.require(:effective_cpd_audit_review).permit(:current_step, :comments,
-            cpd_audit_review_items_attributes: [:id, :item_id, :item_type, :recommendation, :comments]
-          )
-        else
-          raise('unexpected step')
-        end
-      end
-    end
-
   end
 end

data/app/controllers/effective/cpd_audits_controller.rb

@@ -4,9 +4,11 @@ module Effective
 
     include Effective::WizardController
 
-    resource_scope do
-      Effective::CpdAudit.deep.where(user: current_user)
-    end
+    resource_scope -> { EffectiveCpd.CpdAudit.deep.where(user: current_user) }
+
+    submit :resubmit, 'Resubmit Audit'
+
+    page_title 'Chat', only: [:chat]
 
     # Reuse the same view for all cpd_audit_level_section steps
     # https://github.com/zombocom/wicked/blob/v1.3.4/lib/wicked/controller/concerns/render_redirect.rb#L32
@@ -15,43 +17,5 @@ module Effective
       render('cpd_audit_level_section', options)
     end
 
-    def permitted_params
-      case step
-      when :start
-        params.require(:effective_cpd_audit).permit(:current_step)
-      when :information
-        params.require(:effective_cpd_audit).permit(:current_step)
-      when :instructions
-        params.require(:effective_cpd_audit).permit(:current_step)
-      when :conflict
-        params.require(:effective_cpd_audit)
-          .permit(:current_step, :conflict_of_interest, :conflict_of_interest_reason)
-      when :exemption
-        params.require(:effective_cpd_audit)
-          .permit(:current_step, :exemption_request, :exemption_request_reason)
-      when :extension
-        params.require(:effective_cpd_audit)
-          .permit(:current_step, :extension_request, :extension_request_date, :extension_request_reason)
-      when :cpd
-        params.require(:effective_cpd_audit).permit(:current_step)
-      when :questionnaire
-        params.require(:effective_cpd_audit).permit(:current_step)
-      when :files
-        params.require(:effective_cpd_audit).permit(:current_step, files: [])
-      when :submit
-        params.require(:effective_cpd_audit).permit(:current_step)
-      when :complete
-        raise('unexpected post to complete')
-      else
-        raise('unexpected step') unless step.to_s.start_with?('section')
-
-        params.require(:effective_cpd_audit).permit(:current_step,
-          cpd_audit_responses_attributes: [
-            :id, :cpd_audit_id, :cpd_audit_level_question_id, :date, :email, :number, :long_answer, :short_answer, :upload_file, cpd_audit_level_question_option_ids: []
-          ]
-        )
-      end
-    end
-
   end
 end

data/app/datatables/admin/effective_cpd_audit_reviews_datatable.rb

@@ -12,10 +12,12 @@ module Admin
       col :id, visible: false
 
       col :cpd_audit
+      col :cpd_audit_level, search: EffectiveCpd.CpdAuditLevel.all.sorted
 
       col :auditee do |cpd_audit_review|
         user = cpd_audit_review.cpd_audit.user
-        link_to(user, edit_admin_user_path(user))
+        url = "/admin/users/#{user.to_param}/edit"
+        link_to(user, url)
       end
 
       col :user, label: 'Audit Reviewer'

data/app/datatables/admin/effective_cpd_audits_datatable.rb

@@ -1,41 +1,41 @@
 module Admin
   class EffectiveCpdAuditsDatatable < Effective::Datatable
+
     filters do
       scope :all
       scope :available, label: 'In Progress'
-      scope :completed
+      scope :done
       scope :waiting_on_admin
       scope :waiting_on_auditee
       scope :waiting_on_reviewers
     end
 
     datatable do
+      order :updated_at
+
       col :id, visible: false
       col :created_at, visible: false
       col :updated_at, visible: false
 
       col :token, visible: false
 
-      col :cpd_audit_level, label: 'Audit Level'
+      col :cpd_audit_level, label: 'Audit Level', search: EffectiveCpd.CpdAuditLevel.sorted.all
 
       col :user, search: :string, label: 'Auditee'
 
       col(:cpd_audit_reviews, label: 'Auditor Reviewers', search: :string) do |cpd_audit|
         cpd_audit.cpd_audit_reviews.map(&:user).map do |user|
           content_tag(:div, class: 'col-resource_item') do
-            if view.respond_to?(:edit_admin_user_path)
-              link_to(user.to_s, edit_admin_user_path(user))
-            else
-              user.to_s
-            end
+            url = view.respond_to?(:edit_admin_user_path) ? edit_admin_user_path(user) : "/admin/users/#{user.to_param}/edit"
+            link_to(user.to_s, url)
           end
         end.join.html_safe
       end.search do |collection, term, column, sql_column|
-        user_klass = Effective::CpdAudit.first.user&.class
+        user_klass = EffectiveCpd.CpdAudit.first.user&.class
 
         if user_klass.present?
           users = Effective::Resource.new(user_klass).search_any(term)
-          cpd_audit_reviews = Effective::CpdAuditReview.where(user_id: users)
+          cpd_audit_reviews = EffectiveCpd.CpdAuditReview.where(user_id: users)
           collection.where(id: cpd_audit_reviews.select('cpd_audit_id AS id'))
         else
           collection.where(id: nil)
@@ -90,6 +90,8 @@ module Admin
       col :extension_request_date, visible: false
       col :extension_request_reason, visible: false
 
+      col :files, visible: false
+
       actions_col
     end
 

data/app/datatables/effective_cpd_available_audit_reviews_datatable.rb

@@ -8,10 +8,10 @@ class EffectiveCpdAvailableAuditReviewsDatatable < Effective::Datatable
 
     col :cpd_audit_level, label: 'Audit'
     col :due_date
-    col :user, label: 'Auditee', action: false
+    col :name, label: 'Auditee', action: false
 
     col :ready_to_review do |cpd_audit|
-      cpd_audit.was_submitted? ? 'Yes' : 'No'
+      cpd_audit.ready_to_review? ? 'Yes' : 'No'
     end
 
     actions_col(actions: []) do |cpd_audit|
@@ -28,9 +28,9 @@ class EffectiveCpdAvailableAuditReviewsDatatable < Effective::Datatable
   collection do
     raise('expected a current_user') unless current_user.present?
 
-    reviews = Effective::CpdAuditReview.available.where(user: current_user)
+    reviews = EffectiveCpd.CpdAuditReview.available.where(user: current_user)
 
-    Effective::CpdAudit.available.includes(:cpd_audit_reviews)
+    EffectiveCpd.CpdAudit.available.includes(:cpd_audit_reviews)
       .where(id: reviews.select('cpd_audit_id as id'))
   end
 

data/app/datatables/effective_cpd_available_audits_datatable.rb

@@ -24,7 +24,7 @@ class EffectiveCpdAvailableAuditsDatatable < Effective::Datatable
 
   collection do
     raise('expected a current_user') unless current_user.present?
-    Effective::CpdAudit.available.where(user: current_user)
+    EffectiveCpd.CpdAudit.available.where(user: current_user)
   end
 
 end

data/app/datatables/effective_cpd_completed_audit_reviews_datatable.rb

@@ -8,7 +8,7 @@ class EffectiveCpdCompletedAuditReviewsDatatable < Effective::Datatable
 
     col :cpd_audit_level, label: 'Audit'
     col :notification_date, label: 'Date of Notification'
-    col :user, label: 'Auditee', action: false
+    col :name, label: 'Auditee', action: false
 
     col :recommendation do |cpd_audit|
       cpd_audit.cpd_audit_reviews.find { |r| r.user_id == current_user.id }.recommendation
@@ -23,9 +23,9 @@ class EffectiveCpdCompletedAuditReviewsDatatable < Effective::Datatable
   collection do
     raise('expected a current_user') unless current_user.present?
 
-    reviews = Effective::CpdAuditReview.completed.where(user: current_user)
+    reviews = EffectiveCpd.CpdAuditReview.completed.where(user: current_user)
 
-    Effective::CpdAudit.includes(:cpd_audit_reviews)
+    EffectiveCpd.CpdAudit.includes(:cpd_audit_reviews)
       .where(id: reviews.select('cpd_audit_id as id'))
   end
 

data/app/datatables/effective_cpd_completed_audits_datatable.rb

@@ -12,13 +12,13 @@ class EffectiveCpdCompletedAuditsDatatable < Effective::Datatable
     col :determination
 
     actions_col(actions: []) do |cpd_audit|
-      dropdown_link_to('Show', effective_cpd.cpd_audit_build_path(cpd_audit, cpd_audit.last_completed_step))
+      dropdown_link_to('Show', effective_cpd.cpd_audit_build_path(cpd_audit, cpd_audit.last_completed_step || :start))
     end
   end
 
   collection do
     raise('expected a current_user') unless current_user.present?
-    Effective::CpdAudit.completed.where(user: current_user)
+    EffectiveCpd.CpdAudit.completed.where(user: current_user)
   end
 
 end

data/app/helpers/effective_cpd_audits_helper.rb

@@ -12,37 +12,5 @@ module EffectiveCpdAuditsHelper
     [['Yes, I would like to request an extension', true], ['No extension request', false]]
   end
 
-  def cpd_audit_summary_text(cpd_audit)
-    case cpd_audit.status
-    when 'opened'
-      "The audit has been opened. The auditee and audit reviewers have been notified. Waiting for the auditee to submit their audit questionnaire."
-    when 'started'
-      "The auditee has begun their audit questionnaire. Waiting for the auditee to submit their audit questionnaire."
-    when 'conflicted'
-      "The auditee has declared a conflict of interest. Waiting for a new reviewer to be assigned or otherwise resolved."
-    when 'conflicted_resolved'
-      "The auditee had declared a conflict of interest. This has been resolved. Waiting for the auditee to submit their audit questionnaire."
-    when 'exemption_requested'
-      "The auditee has requested an exemption. Waiting for request to be granted or denied."
-    when 'exemption_granted'
-      "The exemption request has been granted. This audit may now be closed."
-    when 'exemption_denied'
-      "The exemption request has been denied. The audit will continue. Waiting for the auditee to submit their audit questionnaire."
-    when 'extension_requested'
-      "The auditee has requested an extension. Waiting for request to be granted or denied."
-    when 'extension_granted'
-      "The extension request has been granted. There is a new deadline. Waiting for the auditee to submit their audit questionnaire."
-    when 'extension_denied'
-      "The extension request has been denied. The deadline remains. Waiting for the auditee to submit their audit questionnaire."
-    when 'submitted'
-      "The auditee has submitted their audit questionnaire. Waiting on review."
-    when 'reviewed'
-      "The audit has been reviewed and is ready for the final determination to be made."
-    when 'closed'
-      "This audit has been closed with a final determination. All done."
-    else
-      raise("unexpected cpd audit status: #{cpd_audit.status}")
-    end
-  end
 
 end

data/app/mailers/effective/cpd_mailer.rb

@@ -6,7 +6,7 @@ module Effective
 
     # CPD Audit
     def cpd_audit_opened(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -18,7 +18,7 @@ module Effective
     end
 
     def cpd_audit_conflicted(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.edit_admin_cpd_audit_url(resource))
       @cpd_audit = resource
@@ -30,7 +30,7 @@ module Effective
     end
 
     def cpd_audit_conflict_resolved(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -42,7 +42,7 @@ module Effective
     end
 
     def cpd_audit_exemption_request(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.edit_admin_cpd_audit_url(resource))
       @cpd_audit = resource
@@ -54,7 +54,7 @@ module Effective
     end
 
     def cpd_audit_exemption_denied(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -66,7 +66,7 @@ module Effective
     end
 
     def cpd_audit_exemption_granted(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -78,7 +78,7 @@ module Effective
     end
 
     def cpd_audit_extension_request(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.edit_admin_cpd_audit_url(resource))
       @cpd_audit = resource
@@ -90,7 +90,7 @@ module Effective
     end
 
     def cpd_audit_extension_denied(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -102,7 +102,7 @@ module Effective
     end
 
     def cpd_audit_extension_granted(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -114,7 +114,7 @@ module Effective
     end
 
     def cpd_audit_submitted(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.edit_admin_cpd_audit_url(resource))
       @cpd_audit = resource
@@ -125,8 +125,20 @@ module Effective
       mail(to: mailer_admin, subject: subject, **headers)
     end
 
+    def cpd_audit_missing_info(resource, opts = {})
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
+
+      @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
+      @cpd_audit = resource
+
+      subject = subject_for(__method__, 'CPD Audit Missing Info', resource, opts)
+      headers = headers_for(resource, opts)
+
+      mail(to: resource.user.email, subject: subject, **headers)
+    end
+
     def cpd_audit_reviewed(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.edit_admin_cpd_audit_url(resource))
       @cpd_audit = resource
@@ -138,7 +150,7 @@ module Effective
     end
 
     def cpd_audit_closed(resource, opts = {})
-      raise('expected an Effective::CpdAudit') unless resource.kind_of?(Effective::CpdAudit)
+      raise('expected an EffectiveCpd.CpdAudit') unless resource.kind_of?(EffectiveCpd.CpdAudit)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_url(resource))
       @cpd_audit = resource
@@ -151,7 +163,7 @@ module Effective
 
     # CPD Audit Review
     def cpd_audit_review_opened(resource, opts = {})
-      raise('expected an Effective::CpdAuditReview') unless resource.kind_of?(Effective::CpdAuditReview)
+      raise('expected an EffectiveCpd.CpdAuditReview') unless resource.kind_of?(EffectiveCpd.CpdAuditReview)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_review_url(resource))
       @cpd_audit_review = resource
@@ -163,7 +175,7 @@ module Effective
     end
 
     def cpd_audit_review_ready(resource, opts = {})
-      raise('expected an Effective::CpdAuditReview') unless resource.kind_of?(Effective::CpdAuditReview)
+      raise('expected an EffectiveCpd.CpdAuditReview') unless resource.kind_of?(EffectiveCpd.CpdAuditReview)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.cpd_audit_review_url(resource))
       @cpd_audit_review = resource
@@ -175,7 +187,7 @@ module Effective
     end
 
     def cpd_audit_review_submitted(resource, opts = {})
-      raise('expected an Effective::CpdAuditReview') unless resource.kind_of?(Effective::CpdAuditReview)
+      raise('expected an EffectiveCpd.CpdAuditReview') unless resource.kind_of?(EffectiveCpd.CpdAuditReview)
 
       @assigns = assigns_for(resource).merge(url: effective_cpd.edit_admin_cpd_audit_url(resource.cpd_audit))
       @cpd_audit_review = resource
@@ -189,12 +201,12 @@ module Effective
     protected
 
     def assigns_for(resource)
-      unless resource.kind_of?(Effective::CpdAudit) || resource.kind_of?(Effective::CpdAuditReview)
-        raise('expected an Effective::CpdAudit or Effective::CpdAuditReview')
+      unless resource.kind_of?(EffectiveCpd.CpdAudit) || resource.kind_of?(EffectiveCpd.CpdAuditReview)
+        raise('expected an EffectiveCpd.CpdAudit or EffectiveCpd.CpdAuditReview')
       end
 
-      @cpd_audit = (resource.kind_of?(Effective::CpdAudit) ? resource : resource.cpd_audit)
-      @cpd_audit_review = (resource if resource.kind_of?(Effective::CpdAuditReview))
+      @cpd_audit = (resource.kind_of?(EffectiveCpd.CpdAudit) ? resource : resource.cpd_audit)
+      @cpd_audit_review = (resource if resource.kind_of?(EffectiveCpd.CpdAuditReview))
 
       @auditee = @cpd_audit.user
       @reviewer = @cpd_audit_review.user if @cpd_audit_review.present?
@@ -203,11 +215,12 @@ module Effective
         audit: {
           title: @cpd_audit.to_s,
           level: @cpd_audit.cpd_audit_level.to_s,
-          determination: @cpd_audit.determination.to_s
+          determination: @cpd_audit.determination.to_s,
+          missing_info_reason: @cpd_audit.missing_info_reason.to_s
         },
         auditee: {
-          name: @auditee.to_s,
-          email: @auditee.email
+          name: @cpd_audit.name,
+          email: (@cpd_audit.user.email unless @cpd_audit.anonymous?).to_s
         }
       }
 
@@ -218,8 +231,8 @@ module Effective
             recommendation: @cpd_audit_review.recommendation.to_s
           },
           reviewer: {
-            name: @reviewer.to_s,
-            email: @reviewer.email
+            name: @cpd_audit_review.name,
+            email: (@cpd_audit_review.user.email unless @cpd_audit_review.anonymous?).to_s
           }
         }
       end