Skip to content

Pvgp/se4781/opencraft release/koa.3 esme - #422

Merged
pedrovgp merged 3 commits into
opencraft-release/koa.3-esmefrom
pvgp/se4781/opencraft-release/koa.3-esme
Oct 1, 2021
Merged

Pvgp/se4781/opencraft release/koa.3 esme#422
pedrovgp merged 3 commits into
opencraft-release/koa.3-esmefrom
pvgp/se4781/opencraft-release/koa.3-esme

Conversation

@pedrovgp

@pedrovgp pedrovgp commented Oct 1, 2021

Copy link
Copy Markdown

This is to push to ESME staging instances this PR here #405
Ticket: https://tasks.opencraft.com/browse/SE-4781

The customer required replacing the default list,
which requires deleting these snippet files.
ADD html snippets from overview unit

From here: https://studio.esme-staging.opencraft.hosting/course/course-v1:esme+CT+2021?show=block-v1%3Aesme%2BCT%2B2021%2Btype%40chapter%2Bblock%400d4aefe059784786bd1f8eb850a633b2

The snippets from the overview Unit were added.

ADD snippets from all other subsections

Added snippets from Typical Section and Quiz
CSS lines were removed from all files, as per requested.

CORRECT typo which made template invalid
@pedrovgp
pedrovgp requested a review from jvdm October 1, 2021 16:41
@pedrovgp pedrovgp self-assigned this Oct 1, 2021

@jvdm jvdm left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

  • I tested this: Tested across the previous change submitted to koa.3 branch.
  • I read through the code
  • N/A I checked for accessibility issues
  • Includes documentation. Code comments are in place where resonable.
  • N/A I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

@pedrovgp
pedrovgp merged commit 48e74c9 into opencraft-release/koa.3-esme Oct 1, 2021
@pedrovgp
pedrovgp deleted the pvgp/se4781/opencraft-release/koa.3-esme branch October 1, 2021 23:25
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