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

EWPP-4496: Add Introduction field to various patterns. #1477

Merged
merged 5 commits into from
Aug 5, 2024
Merged

Conversation

intelektron
Copy link
Member

EWPP-4496

Description

Add Introduction field to link lists.

@intelektron intelektron force-pushed the EWPP-4496 branch 2 times, most recently from 8d587f5 to 25e7ee8 Compare July 15, 2024 11:42
Copy link
Contributor

@22Alexandra 22Alexandra left a comment

Choose a reason for hiding this comment

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

We need test coverage for all the patterns that have been changed.
Please check tests/src/Kernel/fixtures/rendering.yml

@@ -19,6 +20,14 @@
</div>
</div>
{% endif %}
{% set _introduction = introduction|default('') %}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this needed? Instead of moving the value from a variable to another, we can just check if introduction is not empty.

Copy link
Member Author

Choose a reason for hiding this comment

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

I just followed the way from other components, I can change it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@intelektron intelektron force-pushed the EWPP-4496 branch 2 times, most recently from f953074 to fe09a9a Compare July 19, 2024 10:11
@intelektron
Copy link
Member Author

We need test coverage for all the patterns that have been changed. Please check tests/src/Kernel/fixtures/rendering.yml

I've added the introduction field to pattern fixtures

@intelektron intelektron force-pushed the EWPP-4496 branch 3 times, most recently from 3f33ce9 to 66ce866 Compare July 23, 2024 10:28
22Alexandra
22Alexandra previously approved these changes Jul 23, 2024
@upchuk upchuk changed the title EWPP-4496: Add Introduction field to link lists. EWPP-4496: Add Introduction field to various patterns. Aug 5, 2024
@upchuk upchuk merged commit bf1253c into 4.x Aug 5, 2024
1 check passed
@upchuk upchuk deleted the EWPP-4496 branch August 5, 2024 12:22
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.

4 participants