From f3f05d77f2e78dffb08926eaad98835a03f73811 Mon Sep 17 00:00:00 2001 From: Abdurrahman Asad <51022010+A-ASAD@users.noreply.github.com> Date: Mon, 21 Mar 2022 11:05:51 +0500 Subject: [PATCH] fix: modify message on discussion provider LTI configuration page (#265) * fix: modify message on discussion provider LTI configuration page * fix: update message --- .../app-config-form/apps/lti/LtiConfigForm.jsx | 14 +++++++++----- .../app-config-form/apps/lti/messages.js | 2 +- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/src/pages-and-resources/discussions/app-config-form/apps/lti/LtiConfigForm.jsx b/src/pages-and-resources/discussions/app-config-form/apps/lti/LtiConfigForm.jsx index fb5cee6d9..c4906032c 100644 --- a/src/pages-and-resources/discussions/app-config-form/apps/lti/LtiConfigForm.jsx +++ b/src/pages-and-resources/discussions/app-config-form/apps/lti/LtiConfigForm.jsx @@ -1,7 +1,7 @@ import React, { useEffect } from 'react'; import PropTypes from 'prop-types'; -import { ensureConfig, getConfig } from '@edx/frontend-platform'; +import { ensureConfig } from '@edx/frontend-platform'; import { getAuthenticatedUser } from '@edx/frontend-platform/auth'; import { FormattedMessage, injectIntl, intlShape } from '@edx/frontend-platform/i18n'; import { Card, Form, MailtoLink } from '@edx/paragon'; @@ -50,9 +50,14 @@ function LtiConfigForm({ onSubmit, intl, formRef }) { const isInvalidConsumerKey = Boolean(touched.consumerKey && errors.consumerKey); const isInvalidConsumerSecret = Boolean(touched.consumerSecret && errors.consumerSecret); const isInvalidLaunchUrl = Boolean(touched.launchUrl && errors.launchUrl); - const supportEmail = getConfig().SUPPORT_EMAIL; const showLTIConfig = user.administrator; const enablePIISharing = false; + const supportEmails = { + Yellowdig: 'learnmore@yellowdig.com', + 'Ed Discussion': 'team@edstem.org', + InScribe: 'hello@inscribeapp.com', + Piazza: 'team@piazza.com', + }; useEffect(() => { dispatch(updateValidationStatus({ hasError: Object.keys(errors).length > 0 })); @@ -67,9 +72,8 @@ function LtiConfigForm({ onSubmit, intl, formRef }) { {...messages.stuffOnlyConfig} values={{ providerName, - platformName: getConfig().SITE_NAME, - supportEmail: supportEmail ? ( - {supportEmail} + supportEmail: supportEmails[providerName] ? ( + {supportEmails[providerName]} ) : ( 'support' ), diff --git a/src/pages-and-resources/discussions/app-config-form/apps/lti/messages.js b/src/pages-and-resources/discussions/app-config-form/apps/lti/messages.js index b052d7dd4..526eae3f6 100644 --- a/src/pages-and-resources/discussions/app-config-form/apps/lti/messages.js +++ b/src/pages-and-resources/discussions/app-config-form/apps/lti/messages.js @@ -41,7 +41,7 @@ const messages = defineMessages({ }, stuffOnlyConfig: { id: 'authoring.discussions.stuffConfig', - defaultMessage: '{providerName} can only be configured by {platformName} administrators. Please contact {supportEmail} to enable this feature. This will require sharing usernames and emails of learners and the course team with {providerName}', + defaultMessage: 'To enable {providerName} for your course, please contact their support team at {supportEmail} to learn more about pricing and usage. To fully configure {providerName} will also require sharing usernames and emails for learners and course team. Please contact your edX project coordinator to enable PII sharing for this course.', }, piiSharing: { id: 'authoring.discussions.piiSharing',