diff --git a/src/alerts/enrollment-alert/messages.js b/src/alerts/enrollment-alert/messages.js index 1d74456b..5860ff72 100644 --- a/src/alerts/enrollment-alert/messages.js +++ b/src/alerts/enrollment-alert/messages.js @@ -18,7 +18,7 @@ const messages = defineMessages({ }, success: { id: 'learning.enrollment.success', - defaultMessage: "You've successfully enrolled in this course!", + defaultMessage: 'You’ve successfully enrolled in this course!', description: 'A message telling the user that their course enrollment was successful.', }, }); diff --git a/src/course-home/dates-banner/messages.js b/src/course-home/dates-banner/messages.js index 5ad06e48..e8f9223f 100644 --- a/src/course-home/dates-banner/messages.js +++ b/src/course-home/dates-banner/messages.js @@ -3,7 +3,7 @@ import { defineMessages } from '@edx/frontend-platform/i18n'; const messages = defineMessages({ 'datesBanner.datesTabInfoBanner.header': { id: 'datesBanner.datesTabInfoBanner.header', - defaultMessage: "We've built a suggested schedule to help you stay on track. ", + defaultMessage: 'We’ve built a suggested schedule to help you stay on track. ', description: 'Strong text in Dates Tab Info Banner', }, 'datesBanner.datesTabInfoBanner.body': { @@ -35,8 +35,8 @@ const messages = defineMessages({ }, 'datesBanner.upgradeToResetBanner.body': { id: 'datesBanner.upgradeToResetBanner.body', - defaultMessage: `which means that you are unable to participate in graded assignments. It looks like you missed - some important deadlines based on our suggested schedule. To complete graded assignments as part of this course + defaultMessage: `which means that you are unable to participate in graded assignments. It looks like you missed + some important deadlines based on our suggested schedule. To complete graded assignments as part of this course and shift the past due assignments into the future, you can upgrade today.`, description: 'Body in Upgrade To Reset Banner', }, @@ -52,7 +52,7 @@ const messages = defineMessages({ }, 'datesBanner.resetDatesBanner.body': { id: 'datesBanner.resetDatesBanner.body', - defaultMessage: `To keep yourself on track, you can update this schedule and shift the past due assignments into + defaultMessage: `To keep yourself on track, you can update this schedule and shift the past due assignments into the future. Don’t worry—you won’t lose any of the progress you’ve made when you shift your due dates.`, description: 'Body in Reset Dates Banner', }, diff --git a/src/courseware/course/celebration/FirstDiscussionCelebrationModal.jsx b/src/courseware/course/celebration/FirstDiscussionCelebrationModal.jsx index e699073b..089d1325 100644 --- a/src/courseware/course/celebration/FirstDiscussionCelebrationModal.jsx +++ b/src/courseware/course/celebration/FirstDiscussionCelebrationModal.jsx @@ -31,7 +31,7 @@ function FirstDiscussionCelebrationModal({ let normativeDataBodyText; // Bucket 2 corresponds to showing normative data in the body text - if (firstDiscussionUserBucket === 0) { + if (firstDiscussionUserBucket === 2) { if (Math.random() > 0.5) { normativeDataBodyText = (

{intl.formatMessage(messages.discussionBodyText1)}

); } else { diff --git a/src/courseware/course/celebration/messages.js b/src/courseware/course/celebration/messages.js index c758d656..ac41564b 100644 --- a/src/courseware/course/celebration/messages.js +++ b/src/courseware/course/celebration/messages.js @@ -15,11 +15,11 @@ const messages = defineMessages({ }, discussionBodyText1: { id: 'learning.celebration.discussionBodyText1', - defaultMessage: "Learners who participate in discussions are 10x more likely to complete their course than those who don't.", + defaultMessage: 'Learners who participate in discussions are 10x more likely to complete their course than those who don’t.', }, discussionBodyText2: { id: 'learning.celebration.discussionBodyText2', - defaultMessage: "Learners who participate in discussions complete 3x as much course content on average vs. those who don't.", + defaultMessage: 'Learners who participate in discussions complete 3x as much course content on average vs. those who don’t.', }, earned: { id: 'learning.celebration.earned', @@ -27,7 +27,7 @@ const messages = defineMessages({ }, emailSubject: { id: 'learning.celebration.emailSubject', - defaultMessage: "I'm on my way to completing {title} online with {platform}!", + defaultMessage: 'I’m on my way to completing {title} online with {platform}!', description: 'Subject when sharing course progress via email', }, firstDiscussionPost: { diff --git a/src/courseware/course/celebration/utils.jsx b/src/courseware/course/celebration/utils.jsx index cac62eca..4b699dbb 100644 --- a/src/courseware/course/celebration/utils.jsx +++ b/src/courseware/course/celebration/utils.jsx @@ -38,7 +38,7 @@ function recordFirstDiscussionCelebration(courseId) { See edx-platform/common/static/common/js/discussion/views/new_post_view.js and edx-platform/cms/static/common/js/discussion/views/discussion_thread_view.js */ // Tell the LMS - // postCelebrationComplete(courseId, {first_discussion: false}); + // postCelebrationComplete(courseId, { first_discussion: false }); // Tell our analytics sendCelebrationSegmentEvent(courseId, 'edx.ui.lms.celebration.first_discussion.opened'); @@ -82,7 +82,7 @@ function shouldCelebrateOnSectionLoad(courseId, sequenceId, unitId, celebrateFir function shouldCelebrateOnDiscussionPost(firstDiscussion, firstDiscussionUserBucket) { // Bucket 0 === Control group which does not get the discussion celebration. // That check can be removed when we stop using the flag as an experiment. - return firstDiscussion && firstDiscussionUserBucket === 0; + return firstDiscussion && firstDiscussionUserBucket !== 0; } export { diff --git a/src/courseware/course/course-exit/messages.js b/src/courseware/course/course-exit/messages.js index c2a2aaaf..0ae90693 100644 --- a/src/courseware/course/course-exit/messages.js +++ b/src/courseware/course/course-exit/messages.js @@ -42,7 +42,7 @@ const messages = defineMessages({ dashboardLink: { id: 'courseExit.dashboardLink', defaultMessage: 'Dashboard', - description: "Link to user's dashboard", + description: 'Link to user’s dashboard', }, downloadButton: { id: 'courseCelebration.downloadButton', @@ -69,7 +69,7 @@ const messages = defineMessages({ linkedinAddToProfileButton: { id: 'courseCelebration.linkedinAddToProfileButton', defaultMessage: 'Add to LinkedIn profile', - description: "Button to add certificate information to the user's LinkedIn profile", + description: 'Button to add certificate information to the user’s LinkedIn profile', }, nextButtonComplete: { id: 'learn.sequence.navigation.complete.button', // for historical reasons @@ -82,7 +82,7 @@ const messages = defineMessages({ profileLink: { id: 'courseExit.profileLink', defaultMessage: 'Profile', - description: "Link to user's profile", + description: 'Link to user’s profile', }, requestCertificateBodyText: { id: 'courseCelebration.requestCertificateBodyText', diff --git a/src/courseware/course/sequence/content-lock/messages.js b/src/courseware/course/sequence/content-lock/messages.js index 872009b1..f38381d3 100644 --- a/src/courseware/course/sequence/content-lock/messages.js +++ b/src/courseware/course/sequence/content-lock/messages.js @@ -8,7 +8,7 @@ const messages = defineMessages({ }, 'learn.contentLock.complete.prerequisite': { id: 'learn.contentLock.complete.prerequisite', - defaultMessage: "You must complete the prerequisite: '{prereqSectionName}' to access this content.", + defaultMessage: 'You must complete the prerequisite: "{prereqSectionName}" to access this content.', description: 'Message shown to indicate which prerequisite the student must complete prior to accessing the locked content. {prereqSectionName} is the name of the prerequisite.', }, 'learn.contentLock.goToSection': {