From 1ac856725c0170cb2e8cd1cd2029f6931db042fb Mon Sep 17 00:00:00 2001 From: Awais Ansari Date: Tue, 15 Aug 2023 18:33:55 +0500 Subject: [PATCH] fix: moved feedback widget behind the env variable --- .env | 2 ++ .env.development | 2 ++ .env.test | 2 ++ src/discussions/discussions-home/FeedbackWrapper.jsx | 5 +++-- src/index.jsx | 2 ++ 5 files changed, 11 insertions(+), 2 deletions(-) diff --git a/.env b/.env index dfe6dc506..273646442 100644 --- a/.env +++ b/.env @@ -20,3 +20,5 @@ SEGMENT_KEY='' SITE_NAME='' USER_INFO_COOKIE_NAME='' SUPPORT_URL='' +LEARNER_FEEDBACK_URL='' +STAFF_FEEDBACK_URL='' diff --git a/.env.development b/.env.development index 81bd9d4a5..ae4b3cee5 100644 --- a/.env.development +++ b/.env.development @@ -21,3 +21,5 @@ SEGMENT_KEY='' SITE_NAME=localhost USER_INFO_COOKIE_NAME='edx-user-info' SUPPORT_URL='https://support.edx.org' +LEARNER_FEEDBACK_URL='' +STAFF_FEEDBACK_URL='' diff --git a/.env.test b/.env.test index 79290ceac..84bc122cd 100644 --- a/.env.test +++ b/.env.test @@ -19,3 +19,5 @@ SEGMENT_KEY='' SITE_NAME='localhost' USER_INFO_COOKIE_NAME='edx-user-info' SUPPORT_URL='https://support.edx.org' +LEARNER_FEEDBACK_URL='' +STAFF_FEEDBACK_URL='' diff --git a/src/discussions/discussions-home/FeedbackWrapper.jsx b/src/discussions/discussions-home/FeedbackWrapper.jsx index 74d668bb0..1627f869a 100644 --- a/src/discussions/discussions-home/FeedbackWrapper.jsx +++ b/src/discussions/discussions-home/FeedbackWrapper.jsx @@ -2,6 +2,7 @@ import { useEffect } from 'react'; import { useSelector } from 'react-redux'; +import { getConfig } from '@edx/frontend-platform'; import { logError } from '@edx/frontend-platform/logging'; import { RequestStatus } from '../../data/constants'; @@ -22,9 +23,9 @@ export default function useFeedbackWrapper() { useEffect(() => { if (configStatus === RequestStatus.SUCCESSFUL) { - let url = '//w.usabilla.com/9e6036348fa1.js'; + let url = getConfig().LEARNER_FEEDBACK_URL; if (isStaff || isUserGroupTA || isCourseAdmin || isCourseStaff) { - url = '//w.usabilla.com/767740a06856.js'; + url = getConfig().STAFF_FEEDBACK_URL; } try { // eslint-disable-next-line no-undef diff --git a/src/index.jsx b/src/index.jsx index c705bb45c..8c5737b01 100755 --- a/src/index.jsx +++ b/src/index.jsx @@ -37,6 +37,8 @@ initialize({ config: () => { mergeConfig({ LEARNING_BASE_URL: process.env.LEARNING_BASE_URL, + LEARNER_FEEDBACK_URL: process.env.LEARNER_FEEDBACK_URL, + STAFF_FEEDBACK_URL: process.env.STAFF_FEEDBACK_URL, }, 'DiscussionsConfig'); }, },