Skip to content

Update ESoCC strings after UX writing review#5819

Open
marcellamaki wants to merge 2 commits intolearningequality:unstablefrom
marcellamaki:update-esocc-strings
Open

Update ESoCC strings after UX writing review#5819
marcellamaki wants to merge 2 commits intolearningequality:unstablefrom
marcellamaki:update-esocc-strings

Conversation

@marcellamaki
Copy link
Copy Markdown
Member

Summary

References

Reviewer guidance

AI usage

tooltip: 'Review flagged submission',
text: 'Needs changes',
tooltip: 'Submission requires changes before approval',
color: theme.red.v_100,
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these are admin-only so not translated, but I updated the langage here for consistency

const infoSection = wrapper.find('.info-section');
expect(infoSection.exists()).toBe(true);
expect(infoSection.text()).toContain(flaggedPrimaryInfo$());
expect(infoSection.text()).toContain(needsChangesPrimaryInfo$());
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed the string as well for consistency

if (!latestSubmissionIsFinished.value) return undefined;

switch (latestSubmissionStatus.value) {
case CommunityLibraryStatus.PENDING:
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed as redundant

:buttonValue="PublishModes.LIVE"
:currentValue="mode"
:description="modeLiveDescription$()"
:description="publishChannelDescription$()"
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed for clarity

}"
>
{{ incompleteResourcesDescription2$() }}
</div>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed as unnecessary

},
versionNotesLabel: {
message: "Describe what's new in this channel version",
message: "Describe what's included in this channel version",
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was per request from Laura

languageRequiredMessage: {
message: 'Language is required',
context: 'Error message when language selection is required but not provided',
},
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reusing "This field is required" common string

@marcellamaki marcellamaki force-pushed the update-esocc-strings branch from 33d17ac to 5450358 Compare April 9, 2026 22:42
needsChangesPrimaryInfo: {
message:
'A previous version was flagged for review. Ensure you have fixed all highlighted issues before submitting.',
'Your previously submitted version needs changes. Make sure you have addressed all feedback before resubmitting.',
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: i slightly revised the wording here because I think "highlighted issues" might be interpreted too literally, and not everything will always be indicated via the UI

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would only suggest to swap feedback with comments.

alreadySubmittedWarningDescription: {
message:
'Please wait for review or make changes and publish a new version before submitting again.',
'Please wait for the channel to be reviewed. You will see a notification in your Studio account after the review is complete.',
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note - it will not be an email, per richard :)

incompatibleLicensesDescription: {
message:
'"{licenseNames}" - this channel cannot be distributed via Kolibri. Please fix licensing before submitting a new version.',
'"{licenseNames}" - this channel cannot be distributed via Kolibri. If you cannot change the license, remove all resources with "{licenseNames}" before submitting again.',
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realized belatedly that this also appears as a warning for "all rights reserved" as well before submission, so I repeated the {licenseNames} so it could be flexible.

@marcellamaki marcellamaki force-pushed the update-esocc-strings branch 2 times, most recently from 3d91548 to f8a260d Compare April 10, 2026 17:29
pdate tests to align with string updates
@marcellamaki marcellamaki force-pushed the update-esocc-strings branch from f8a260d to 13335de Compare April 10, 2026 17:41
@marcellamaki marcellamaki marked this pull request as ready for review April 10, 2026 17:41
publishDraftDescription: {
message:
'Your channel will be saved as a draft, allowing you to review or conduct quality checks without altering the published version on Kolibri public library.',
'Your channel will be saved as a draft, allowing you to review or conduct quality checks without altering the version Kolibri users can access. To see your channel in Kolibri, import using the draft channel token.',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion:

Your channel will be saved as a draft, allowing you to review and do quality checks on the draft, without altering the current version available for Kolibri users. To see this draft channel in Kolibri, import using the draft channel token.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants