Skip to content

[SE-5135] Update HTML templates to the v2 counterparts. - #453

Merged
keithgg merged 2 commits into
opencraft-release/koa.3-esmefrom
keith/se_5135_esme_update_custom_html_templates
Jan 25, 2022
Merged

[SE-5135] Update HTML templates to the v2 counterparts.#453
keithgg merged 2 commits into
opencraft-release/koa.3-esmefrom
keith/se_5135_esme_update_custom_html_templates

Conversation

@keithgg

@keithgg keithgg commented Jan 21, 2022

Copy link
Copy Markdown

Description

For Studio users updates templates to the v2 counterparts.

Screenshot_20220121_115125
Screenshot_20220121_115148

Supporting information

Testing instructions

  • While editing a section in the studio, click on the HTML block.
  • You will see the list of templates in this commit. Eg. Calendar.

Deadline

21 Jan 2022.

@keithgg
keithgg requested a review from pkulkark January 21, 2022 09:53
@keithgg

keithgg commented Jan 21, 2022

Copy link
Copy Markdown
Author

@pkulkark I deleted the templates that weren't mentioned in the v2 section. Should I have kept them?

@pkulkark pkulkark left a comment

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.

@keithgg Nice work! Just a couple of nits to address. Otherwise looking good.

metadata:
display_name: Downloads
data: |
<div class="div-info"><img class="icon-image" src="/static/!downloads.png" alt="" />

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.

Nit: I'm guessing the "!" in the src field got in there by mistake but it's preventing the image from loading currectly. So please correct it.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks! Fixed.

<p>Add text and links to your optional readings here.</p>
</div>
</div> No newline at end of file
display_name: Optional Reading

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.

Nit: The display name also need to be updated to "Optional Reading or Optional Resources".

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Nice one. Sorted

@pkulkark pkulkark left a comment

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.

Thanks for making the changes @keithgg. LGTM 👍

  • I tested this: verified the templates are updates as requested by Esme.
  • I read through the code

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