Skip to content

Conversation

@ciampo
Copy link
Contributor

@ciampo ciampo commented Oct 3, 2024

@ciampo ciampo requested a review from ajitbohra as a code owner October 3, 2024 11:02
@ciampo ciampo requested a review from a team October 3, 2024 11:02
@ciampo ciampo self-assigned this Oct 3, 2024
@github-actions
Copy link

github-actions bot commented Oct 3, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: ciampo <[email protected]>
Co-authored-by: mirka <[email protected]>
Co-authored-by: tyxla <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@ciampo ciampo added [Type] Bug An existing feature does not function as intended [Package] Components /packages/components labels Oct 3, 2024
@ciampo
Copy link
Contributor Author

ciampo commented Oct 3, 2024

CI errors are caused by #65839 (comment) — I will rebase once the target branch has them solved.

Update: fix was added, I rebased to include it.

* Composite: fix legacy implementation passing store prop

* Simplify code

* Improve comment

* Apply same fix to the top level Composite too

* CHANGELOG

---

Co-authored-by: ciampo <[email protected]>
Co-authored-by: tyxla <[email protected]>
@ciampo ciampo requested a review from mirka October 3, 2024 13:12
Copy link
Member

@tyxla tyxla left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Components /packages/components [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants