From 1c003efd621afbf3560eaa01543b200d5cbaa12a Mon Sep 17 00:00:00 2001 From: Mehak Nasir Date: Sat, 18 Dec 2021 00:23:13 +0500 Subject: [PATCH] fix: pii sharing fields values are fixed in accordance with backend flag --- .../discussions/app-config-form/apps/lti/LtiConfigForm.jsx | 4 ++-- src/pages-and-resources/discussions/data/api.js | 2 +- 2 files changed, 3 insertions(+), 3 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 5e49e5bd3..9bbed83f2 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 @@ -31,8 +31,8 @@ function LtiConfigForm({ onSubmit, intl, formRef }) { consumerKey: appConfig?.consumerKey || '', consumerSecret: appConfig?.consumerSecret || '', launchUrl: appConfig?.launchUrl || '', - piiShareUsername: piiConfig.piiShareUsername, - piiShareEmail: piiConfig.piiShareEmail, + piiShareUsername: piiConfig.piiSharing ? piiConfig.piiShareUsername : undefined, + piiShareEmail: piiConfig.piiSharing ? piiConfig.piiShareEmail : undefined, }; const user = getAuthenticatedUser(); const { externalLinks } = app; diff --git a/src/pages-and-resources/discussions/data/api.js b/src/pages-and-resources/discussions/data/api.js index 3d08a4e09..ebf0b1bb3 100644 --- a/src/pages-and-resources/discussions/data/api.js +++ b/src/pages-and-resources/discussions/data/api.js @@ -72,7 +72,7 @@ function normalizePluginConfig(data) { function normalizePiiSharing(data) { return { - piiSharing: 'pii_share_username' in data || 'pii_share_email' in data, + piiSharing: data.pii_sharing_allowed, piiShareUsername: data.pii_share_username, piiShareEmail: data.pii_share_email, };