+
{{ $t('REPORT.BUSINESS_HOURS') }}
@@ -105,6 +108,10 @@ export default {
type: Boolean,
default: false,
},
+ showBusinessHoursSwitch: {
+ type: Boolean,
+ default: true,
+ },
},
data() {
return {
diff --git a/app/javascript/dashboard/routes/dashboard/settings/reports/components/WootReports.vue b/app/javascript/dashboard/routes/dashboard/settings/reports/components/WootReports.vue
index 6d8b49c62..34188a12a 100644
--- a/app/javascript/dashboard/routes/dashboard/settings/reports/components/WootReports.vue
+++ b/app/javascript/dashboard/routes/dashboard/settings/reports/components/WootReports.vue
@@ -61,6 +61,7 @@ import format from 'date-fns/format';
import { GROUP_BY_FILTER, METRIC_CHART } from '../constants';
import reportMixin from '../../../../../mixins/reportMixin';
import { formatTime } from '@chatwoot/utils';
+import { generateFileName } from '../../../../../helper/downloadHelper';
const REPORTS_KEYS = {
CONVERSATIONS: 'conversations_count',
@@ -250,26 +251,17 @@ export default {
});
},
downloadReports() {
- const { from, to } = this;
- const fileName = `${this.type}-report-${format(
- fromUnixTime(to),
- 'dd-MM-yyyy'
- )}.csv`;
- switch (this.type) {
- case 'agent':
- this.$store.dispatch('downloadAgentReports', { from, to, fileName });
- break;
- case 'label':
- this.$store.dispatch('downloadLabelReports', { from, to, fileName });
- break;
- case 'inbox':
- this.$store.dispatch('downloadInboxReports', { from, to, fileName });
- break;
- case 'team':
- this.$store.dispatch('downloadTeamReports', { from, to, fileName });
- break;
- default:
- break;
+ const { from, to, type } = this;
+ const dispatchMethods = {
+ agent: 'downloadAgentReports',
+ label: 'downloadLabelReports',
+ inbox: 'downloadInboxReports',
+ team: 'downloadTeamReports',
+ };
+ if (dispatchMethods[type]) {
+ const fileName = generateFileName({ type, to });
+ const params = { from, to, fileName };
+ this.$store.dispatch(dispatchMethods[type], params);
}
},
changeSelection(index) {
diff --git a/app/javascript/dashboard/store/modules/csat.js b/app/javascript/dashboard/store/modules/csat.js
index 7bc1dad6d..2557e2698 100644
--- a/app/javascript/dashboard/store/modules/csat.js
+++ b/app/javascript/dashboard/store/modules/csat.js
@@ -1,6 +1,7 @@
import * as MutationHelpers from 'shared/helpers/vuex/mutationHelpers';
import types from '../mutation-types';
import CSATReports from '../../api/csatReports';
+import { downloadCsvFile } from '../../helper/downloadHelper';
const computeDistribution = (value, total) =>
((value * 100) / total).toFixed(2);
@@ -107,6 +108,11 @@ export const actions = {
commit(types.SET_CSAT_RESPONSE_UI_FLAG, { isFetchingMetrics: false });
}
},
+ downloadCSATReports(_, params) {
+ return CSATReports.download(params).then(response => {
+ downloadCsvFile(params.fileName, response.data);
+ });
+ },
};
export const mutations = {
diff --git a/app/javascript/dashboard/store/modules/reports.js b/app/javascript/dashboard/store/modules/reports.js
index dc9a6c33b..9ae571803 100644
--- a/app/javascript/dashboard/store/modules/reports.js
+++ b/app/javascript/dashboard/store/modules/reports.js
@@ -5,7 +5,7 @@ import * as types from '../mutation-types';
import Report from '../../api/reports';
import Vue from 'vue';
-import { downloadCsvFile } from '../../helper/downloadCsvFile';
+import { downloadCsvFile } from '../../helper/downloadHelper';
const state = {
fetchingStatus: false,
diff --git a/app/policies/csat_survey_response_policy.rb b/app/policies/csat_survey_response_policy.rb
index c0ce8821b..afcce00e9 100644
--- a/app/policies/csat_survey_response_policy.rb
+++ b/app/policies/csat_survey_response_policy.rb
@@ -6,4 +6,8 @@ class CsatSurveyResponsePolicy < ApplicationPolicy
def metrics?
@account_user.administrator?
end
+
+ def download?
+ @account_user.administrator?
+ end
end
diff --git a/app/views/api/v1/accounts/csat_survey_responses/download.csv.erb b/app/views/api/v1/accounts/csat_survey_responses/download.csv.erb
new file mode 100644
index 000000000..8b694ed35
--- /dev/null
+++ b/app/views/api/v1/accounts/csat_survey_responses/download.csv.erb
@@ -0,0 +1,38 @@
+<%=
+ CSV.generate_line([
+ I18n.t('reports.csat.headers.agent_name'),
+ I18n.t('reports.csat.headers.rating'),
+ I18n.t('reports.csat.headers.feedback'),
+ I18n.t('reports.csat.headers.contact_name'),
+ I18n.t('reports.csat.headers.contact_email_address'),
+ I18n.t('reports.csat.headers.contact_phone_number'),
+ I18n.t('reports.csat.headers.link_to_the_conversation'),
+ I18n.t('reports.csat.headers.recorded_at')
+ ])
+-%>
+<% @csat_survey_responses.each do |csat_response| %>
+<% assigned_agent = csat_response.assigned_agent %>
+<% contact = csat_response.contact %>
+<% conversation = csat_response.conversation %>
+<%=
+ CSV.generate_line([
+ assigned_agent ? "#{assigned_agent.name} (#{assigned_agent.email})" : nil,
+ csat_response.rating,
+ csat_response.feedback_message.present? ? csat_response.feedback_message : nil,
+ contact&.name.present? ? contact&.name: nil,
+ contact&.email.present? ? contact&.email: nil,
+ contact&.phone_number.present? ? contact&.phone_number: nil,
+ conversation ? app_account_conversation_url(account_id: Current.account.id, id: conversation.display_id): nil,
+ csat_response.created_at,
+ ])
+-%>
+<% end %>
+<%=
+ CSV.generate_line([
+ I18n.t(
+ 'reports.period',
+ since: Date.strptime(params[:since], '%s'),
+ until: Date.strptime(params[:until], '%s')
+ )
+ ])
+-%>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 0c92c32eb..bd7829b53 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -60,6 +60,16 @@ en:
avg_first_response_time: Avg first response time (Minutes)
avg_resolution_time: Avg resolution time (Minutes)
default_group_by: day
+ csat:
+ headers:
+ contact_name: Contact Name
+ contact_email_address: Contact Email Address
+ contact_phone_number: Contact Phone Number
+ link_to_the_conversation: Link to the conversation
+ agent_name: Agent Name
+ rating: Rating
+ feedback: Feedback Comment
+ recorded_at: Recorded date
notifications:
notification_title:
diff --git a/config/routes.rb b/config/routes.rb
index aa23be1e7..a80f6ca39 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -106,6 +106,7 @@ Rails.application.routes.draw do
resources :csat_survey_responses, only: [:index] do
collection do
get :metrics
+ get :download
end
end
resources :custom_attribute_definitions, only: [:index, :show, :create, :update, :destroy]
diff --git a/spec/controllers/api/v1/accounts/csat_survey_responses_controller_spec.rb b/spec/controllers/api/v1/accounts/csat_survey_responses_controller_spec.rb
index abb4231d9..f431f0ed5 100644
--- a/spec/controllers/api/v1/accounts/csat_survey_responses_controller_spec.rb
+++ b/spec/controllers/api/v1/accounts/csat_survey_responses_controller_spec.rb
@@ -148,4 +148,38 @@ RSpec.describe 'CSAT Survey Responses API', type: :request do
end
end
end
+
+ describe 'GET /api/v1/accounts/{account.id}/csat_survey_responses/download' do
+ context 'when it is an unauthenticated user' do
+ it 'returns unauthorized' do
+ get "/api/v1/accounts/#{account.id}/csat_survey_responses/download"
+ expect(response).to have_http_status(:unauthorized)
+ end
+ end
+
+ context 'when it is an authenticated user' do
+ let(:params) { { since: 5.days.ago.to_time.to_i.to_s, until: Time.zone.tomorrow.to_time.to_i.to_s } }
+
+ it 'returns unauthorized for agents' do
+ get "/api/v1/accounts/#{account.id}/csat_survey_responses/download",
+ params: params,
+ headers: agent.create_new_auth_token
+
+ expect(response).to have_http_status(:unauthorized)
+ end
+
+ it 'returns summary' do
+ get "/api/v1/accounts/#{account.id}/csat_survey_responses/download",
+ params: params,
+ headers: administrator.create_new_auth_token
+
+ expect(response).to have_http_status(:success)
+
+ content = CSV.parse(response.body)
+ # Check rating from CSAT Row
+ expect(content[1][1]).to eq '1'
+ expect(content.length).to eq 3
+ end
+ end
+ end
end