Skip to content

Conversation

@Paul-Hebert
Copy link
Contributor

Overview

The Comment template has a replies block allowing devs to customize how replies are output.

This block was not working correctly because it was nested inside another embed.

This PR uses set to define the replies outside of the embed and then pass them in.

The Comment template has a `replies` block allowing devs to customize how replies are output.

This block was not working correctly because it was nested inside another embed.

This PR uses `set` to define the replies outside of the embed and then pass them in.
@Paul-Hebert Paul-Hebert requested review from a team and tylersticka July 23, 2021 22:07
@changeset-bot
Copy link

changeset-bot bot commented Jul 23, 2021

🦋 Changeset detected

Latest commit: efd3e74

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cloudfour/patterns Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Paul-Hebert Paul-Hebert marked this pull request as ready for review July 23, 2021 22:07
} only %}
{% endfor %}
{% endblock %}
{% endset %}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm open to other ideas if there's a less yucky way to handle this. This needs to be defined after _child_heading_depth is set

Copy link
Member

Choose a reason for hiding this comment

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

I think this is fine 👍

} only %}
{% endfor %}
{% endblock %}
{% endset %}
Copy link
Member

Choose a reason for hiding this comment

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

I think this is fine 👍

@Paul-Hebert Paul-Hebert merged commit 7463bd6 into v-next Jul 23, 2021
@Paul-Hebert Paul-Hebert deleted the reply-block-issue branch July 23, 2021 22:26
@github-actions github-actions bot mentioned this pull request Jul 23, 2021
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.

3 participants