fix: display same message for everyone during discussions provider configuration (#257)

This commit is contained in:
Awais Ansari
2022-02-23 11:25:56 +05:00
committed by GitHub
parent 6389847310
commit 4b2b599eaf
2 changed files with 15 additions and 21 deletions

View File

@@ -62,9 +62,22 @@ function LtiConfigForm({ onSubmit, intl, formRef }) {
<Card className="mb-5 p-5" data-testid="ltiConfigForm">
<Form ref={formRef} onSubmit={handleSubmit}>
<h3 className="mb-3">{providerName}</h3>
{showLTIConfig ? (
<p>
<FormattedMessage
{...messages.stuffOnlyConfig}
values={{
providerName,
platformName: getConfig().SITE_NAME,
supportEmail: supportEmail ? (
<MailtoLink to={supportEmail}>{supportEmail}</MailtoLink>
) : (
'support'
),
}}
/>
</p>
{showLTIConfig && (
<>
<p>{intl.formatMessage(messages.adminOnlyConfig, { providerName })}</p>
<p>{intl.formatMessage(messages.formInstructions)}</p>
<Form.Group
controlId="consumerKey"
@@ -115,21 +128,6 @@ function LtiConfigForm({ onSubmit, intl, formRef }) {
)}
</Form.Group>
</>
) : (
<p>
<FormattedMessage
{...messages.stuffOnlyConfig}
values={{
providerName,
platformName: getConfig().SITE_NAME,
supportEmail: supportEmail ? (
<MailtoLink to={supportEmail}>{supportEmail}</MailtoLink>
) : (
'support'
),
}}
/>
</p>
)}
{(enablePIISharing) && (
<div data-testid="piiSharingFields">

View File

@@ -43,10 +43,6 @@ const messages = defineMessages({
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}',
},
adminOnlyConfig: {
id: 'authoring.discussions.adminOnlyConfig',
defaultMessage: 'This configuration will require sharing usernames and emails of learners and the course team with {providerName}',
},
piiSharing: {
id: 'authoring.discussions.piiSharing',
defaultMessage: 'Optionally share a user\'s username and/or email with the LTI provider:',