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

Email editor – fix sidebar header [MAILPOET-6371] #5986

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

costasovo
Copy link
Contributor

@costasovo costasovo commented Dec 9, 2024

Description

This PR fixes the issue that is when the new Gutenberg version is active.
The issue was already fixed upstream WordPress/gutenberg#64474
Because @wordpress/interface is not distributed with WP Core, we bundle it. We were using an older version. This PR fixes the issue by updating to the latest version.

Code review notes

N/A

QA notes

N/A

Linked PRs

N/A

Linked tickets

MAILPOET-6371

After-merge notes

N/A

Tasks

  • I followed best practices for translations
  • I added sufficient test coverage
  • I embraced TypeScript by either creating new files in TypeScript or converting existing JavaScript files when making changes

Preview

Preview in WordPress Playground

The latest successful build from email-editor/fix-sidebar-header will be used. If none is available, the link won't work.

The issue was fixed in @wordpress/interface but we were using
the older version.
The @wordpress/interface is not distributed standalone in WP Core
so we bundle it.
[MAILPOET-6371]
@costasovo costasovo requested a review from lysyjan December 9, 2024 13:23
Copy link
Contributor

@lysyjan lysyjan left a comment

Choose a reason for hiding this comment

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

Good job @costasovo with investigating this issue! Thank you

@lysyjan lysyjan merged commit c35bf6a into trunk Dec 9, 2024
28 checks passed
@lysyjan lysyjan deleted the email-editor/fix-sidebar-header branch December 9, 2024 13:47
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.

2 participants