Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Avoid auto-escaping in image component #1108

Merged
merged 7 commits into from
Apr 17, 2019

Conversation

remydenton
Copy link
Collaborator

Jira

http://vjira2:8080/browse/WWWD-3263

Summary

Prevents autoescaping of image contents.

Details

See http://vjira2:8080/browse/WWWD-3263 for the backstory.

How to test

Copy the changes made in this PR to node_modules/@bolt/components-image/src/image.twig in the pega.com Drupal theme. Before these updates, images will be missing; after these updates, they'll be restored.

@remydenton remydenton requested a review from sghoweri March 12, 2019 21:08
@remydenton remydenton force-pushed the hotfix/WWWD-3263-prevent-autoescaping branch from 76eb4fa to b27b15f Compare March 14, 2019 17:40
@sghoweri sghoweri requested a review from danielamorse April 17, 2019 11:40
@sghoweri
Copy link
Contributor

@danielamorse I'm going to get this merged into the v2.x branch so I can cut a v2.3.1 release however I wanted to make sure you get a chance to look through this separately since you've been neck deep in the image component refactor + the fact that this issue could still happen again in other places... 😅

Copy link
Contributor

@sghoweri sghoweri left a comment

Choose a reason for hiding this comment

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

Still double checking the dockerfile bit of this but I'm fine oking the Twig template updates till we can do some further digging

@sghoweri sghoweri merged commit 5d813eb into release/2.x Apr 17, 2019
@sghoweri sghoweri deleted the hotfix/WWWD-3263-prevent-autoescaping branch April 17, 2019 17:29
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