-
Notifications
You must be signed in to change notification settings - Fork 32
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
Conversation
8d587f5
to
25e7ee8
Compare
There was a problem hiding this 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('') %} |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
f953074
to
fe09a9a
Compare
I've added the introduction field to pattern fixtures |
3f33ce9
to
66ce866
Compare
EWPP-4496
Description
Add Introduction field to link lists.