Add indication to admin UI of whether a report has been forwarded (#13237)

* Add indication to admin UI of whether a report has been forwarded

* Rework how forwarded status is displayed

Co-authored-by: Claire <claire.github-309c@sitedethib.com>
This commit is contained in:
ThibG 2020-12-15 04:30:15 +01:00 committed by GitHub
parent 7c800fa801
commit 1390cc194b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
7 changed files with 25 additions and 1 deletions

View file

@ -14,6 +14,7 @@
# target_account_id :bigint(8) not null # target_account_id :bigint(8) not null
# assigned_account_id :bigint(8) # assigned_account_id :bigint(8)
# uri :string # uri :string
# forwarded :boolean
# #
class Report < ApplicationRecord class Report < ApplicationRecord

View file

@ -24,7 +24,8 @@ class ReportService < BaseService
target_account: @target_account, target_account: @target_account,
status_ids: @status_ids, status_ids: @status_ids,
comment: @comment, comment: @comment,
uri: @options[:uri] uri: @options[:uri],
forwarded: ActiveModel::Type::Boolean.new.cast(@options[:forward])
) )
end end

View file

@ -59,6 +59,10 @@
= fa_icon('camera') = fa_icon('camera')
= report.media_attachments.count = report.media_attachments.count
- if report.forwarded?
·
= t('admin.reports.forwarded_to', domain: target_account.domain)
.report-card__summary__item__assigned .report-card__summary__item__assigned
- if report.assigned_account.present? - if report.assigned_account.present?
= admin_account_link_to report.assigned_account = admin_account_link_to report.assigned_account

View file

@ -46,6 +46,16 @@
%td{ colspan: 2 } %td{ colspan: 2 }
- if @report.action_taken? - if @report.action_taken?
= table_link_to 'envelope-open', t('admin.reports.reopen'), admin_report_path(@report, outcome: 'reopen'), method: :put = table_link_to 'envelope-open', t('admin.reports.reopen'), admin_report_path(@report, outcome: 'reopen'), method: :put
- unless @report.target_account.local?
%tr
%th= t('admin.reports.forwarded')
%td{ colspan: 3 }
- if @report.forwarded.nil?
\-
- elsif @report.forwarded?
= t('simple_form.yes')
- else
= t('simple_form.no')
- if !@report.action_taken_by_account.nil? - if !@report.action_taken_by_account.nil?
%tr %tr
%th= t('admin.reports.action_taken_by') %th= t('admin.reports.action_taken_by')

View file

@ -517,6 +517,8 @@ en:
comment: comment:
none: None none: None
created_at: Reported created_at: Reported
forwarded: Forwarded
forwarded_to: Forwarded to %{domain}
mark_as_resolved: Mark as resolved mark_as_resolved: Mark as resolved
mark_as_unresolved: Mark as unresolved mark_as_unresolved: Mark as unresolved
notes: notes:

View file

@ -0,0 +1,5 @@
class AddForwardedToReports < ActiveRecord::Migration[5.2]
def change
add_column :reports, :forwarded, :boolean
end
end

View file

@ -717,6 +717,7 @@ ActiveRecord::Schema.define(version: 2020_12_06_004238) do
t.bigint "target_account_id", null: false t.bigint "target_account_id", null: false
t.bigint "assigned_account_id" t.bigint "assigned_account_id"
t.string "uri" t.string "uri"
t.boolean "forwarded"
t.index ["account_id"], name: "index_reports_on_account_id" t.index ["account_id"], name: "index_reports_on_account_id"
t.index ["target_account_id"], name: "index_reports_on_target_account_id" t.index ["target_account_id"], name: "index_reports_on_target_account_id"
end end