Skip to content

Commit

Permalink
chore: PR feedback 2
Browse files Browse the repository at this point in the history
  • Loading branch information
brobro10000 committed Jun 12, 2024
1 parent ecc52d6 commit 8342bd9
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 35 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -95,11 +95,8 @@ const DeleteArchivedCoursesDialogs = ({
<p>
<FormattedMessage
id="highlights.highlights.tab.archived.courses.modal.message"
defaultMessage="Deleting the archived courses in this highlight will remove it from your learners' {doubleQuote}Find a Course{doubleQuote}. This action is permanent and cannot be undone."
defaultMessage={"Deleting the archived courses in this highlight will remove it from your learners' \"Find a Course\". This action is permanent and cannot be undone."}
description="Message shown in the modal to delete archived courses."
values={{
doubleQuote: '"',
}}
/>
</p>
</ModalDialog.Body>
Expand Down
5 changes: 1 addition & 4 deletions src/components/ContentHighlights/DeleteHighlightSet.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -172,11 +172,8 @@ const DeleteHighlightSet = ({ enterpriseId, enterpriseSlug }) => {
<p>
<FormattedMessage
id="highlights.modal.delete.highlight.confirmation.message"
defaultMessage="Deleting this highlight will remove it from your learners' {doubleQuote}Find a Course{doubleQuote}. This action is permanent and cannot be undone."
defaultMessage={"Deleting this highlight will remove it from your learners' \"Find a Course\". This action is permanent and cannot be undone."}
description="Confirmation message shown when deleting a highlight."
values={{
doubleQuote: '"',
}}
/>
</p>
</AlertModal>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,14 +32,8 @@ const HighlightStepperTitle = () => (
<strong>
<FormattedMessage
id="highlights.new.highlights.stepper.stepper.step.pro.tip.text.create.title"
defaultMessage="Pro tip: we recommend naming your highlight collection to reflect skills
it aims to develop, or to draw the attention of specific groups it targets.
For example, {doubleQuote}Recommended for Marketing{doubleQuote}
or {doubleQuote}Develop Leadership Skills{doubleQuote}."
defaultMessage={'Pro tip: we recommend naming your highlight collection to reflect skills it aims to develop, or to draw the attention of specific groups it targets. For example, "Recommended for Marketing" or "Develop Leadership Skills".'}
description="Create title pro tip message shown to administrators during creation of new content highlights"
values={{
doubleQuote: '"',
}}
/>
</strong>
</p>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,10 @@ const AssignmentEnrollByDateCell = ({ row }) => {
const { original: { earliestPossibleExpiration: { date } } } = row;

const formattedEnrollByDate = formatDate(date);
const isAssignmentExpiringSoon = isTodayWithinDateThreshold(
{
days: ENROLL_BY_DATE_DAYS_THRESHOLD,
date,
},
);
const isAssignmentExpiringSoon = isTodayWithinDateThreshold({
days: ENROLL_BY_DATE_DAYS_THRESHOLD,
date,
});

return (
<Stack direction="horizontal" gap={1}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,8 @@ const CancelAssignmentModal = ({
<p>
<FormattedMessage
id="lcm.budget.detail.page.catalog.tab.cancel.assignment.modal.body2"
defaultMessage="The learner will be notified that you have canceled the assignment. The funds associated with this course assignment will move from {doubleQuote}assigned{doubleQuote} back to {doubleQuote}available{doubleQuote}."
defaultMessage={'The learner will be notified that you have canceled the assignment. The funds associated with this course assignment will move from "assigned" back to "available".'}
description="Body text for the cancel assignment modal which informs the user that the learner will be notified that the assignment has been canceled."
values={{ doubleQuote: '"' }}
/>
</p>
</ModalDialog.Body>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,8 @@ const RemindAssignmentModal = ({
<p>
<FormattedMessage
id="lcm.budget.detail.page.catalog.tab.remind.assignment.modal.body2"
defaultMessage="When your learner completes enrollment, the associated {doubleQuote}assigned{doubleQuote} funds will be marked as {doubleQuote}spent{doubleQuote}."
defaultMessage={'When your learner completes enrollment, the associated "assigned" funds will be marked as "spent".'}
description="Text2 for the body of the remind assignment modal"
values={{ doubleQuote: '"' }}
/>
</p>
</ModalDialog.Body>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -78,23 +78,15 @@ const AssignmentAllocationHelpCollapsibles = ({ enterpriseId, course }) => (
<li>
<FormattedMessage
id="lcm.budget.detailsPage.catalog.tab.course.card.total.assignment.cost"
defaultMessage="The total assignment cost will be earmarked as {doubleQuote}assigned{doubleQuote} funds in your
Learner Credit budget so you can't overspend."
defaultMessage={"The total assignment cost will be earmarked as \"assigned\" funds in your Learner Credit budget so you can't overspend."}
description="A step which explains that the total assignment cost will be earmarked as 'assigned' funds in your Learner Credit budget"
values={{
doubleQuote: '"',
}}
/>
</li>
<li>
<FormattedMessage
id="lcm.budget.detail.page.catalog.tab.course.card.course.cost.will.convert"
defaultMessage="The course cost will automatically convert from {doubleQuote}assigned{doubleQuote} to {doubleQuote}spent{doubleQuote} funds
when your learners complete registration."
defaultMessage={'The course cost will automatically convert from "assigned" to "spent" funds when your learners complete registration.'}
description="A step which explains that the course cost will automatically convert from 'assigned' to 'spent' funds when learners complete registration"
values={{
doubleQuote: '"',
}}
/>
</li>
</ul>
Expand Down

0 comments on commit 8342bd9

Please sign in to comment.