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

DS-839: Fix broken schema to fix full_bleed prop #2538

Merged
merged 1 commit into from Sep 6, 2022

Conversation

danielamorse
Copy link
Collaborator

Jira

https://pegadigitalit.atlassian.net/browse/DS-839

Summary

Fix broken schema to fix full_bleed prop.

Details

In 5.7.1 we added multiple schemas to Profile but did not update the original profile.twig file to reference the schema. It now points to the schema properly and the full_bleed prop works again.

How to test

@github-actions github-actions bot added the type: bugfix List this PR in the 'Bug Fixes' section of the release notes. label Sep 6, 2022
@colbytcook colbytcook had a problem deploying to bugfix/DS-839-fix-full-bleed--5466daf--commit-preview September 6, 2022 14:14 Failure
Copy link
Collaborator

@MarcinMr MarcinMr left a comment

Choose a reason for hiding this comment

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

Seems like full-bleed works properly. The photo is aligned left on desktops. On tablets and mobiles is centered. It looks well in relation to these mockups: https://pegadigitalit.atlassian.net/browse/DS-679

@danielamorse danielamorse merged commit 015a4f6 into master Sep 6, 2022
@danielamorse danielamorse deleted the bugfix/DS-839-fix-full-bleed branch September 6, 2022 17:04
danielamorse added a commit that referenced this pull request Sep 7, 2022
…-bleed

DS-839: Fix broken schema to fix full_bleed prop
@danielamorse danielamorse mentioned this pull request Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bugfix List this PR in the 'Bug Fixes' section of the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants