-
Notifications
You must be signed in to change notification settings - Fork 384
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
Gutenberg: Fix displaying validation warning and usage of PHP function #1612
Conversation
Sorry, Will Review Tomorrow Hi @miina, |
@kienstra Of course! |
Sorry For The Delay Hi @miina, |
Looks Good, Request For Review Hi @westonruter, @miina's PR looks good, and it fixes the issue as I understand it. It would be great to have this PR in Here's a feature branch with Miina's commits cherry-picked onto a branch off of BeforeAfter |
I'm re-basing the commits here onto the |
6f05668
to
34301c1
Compare
|
||
module.validationWarningNoticeId = wp.data.dispatch( 'core/editor' ).createWarningNotice( noticeElement, { spokenMessage: noticeMessage } ).notice.id; | ||
wp.data.dispatch( 'core/notices' ).createNotice( 'warning', noticeMessage, noticeOptions ); |
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.
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.
@westonruter Weird, that would indicate as if core/notices
wouldn't exist. Is this with the latest version of Gutenberg?
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.
Humm. I'm not seeing the issue anymore. I opened an incognito window and everything. I'll report if I see it again.
Makes use of
core/notices
. Updates creating a warning notice.See #1604.