-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
DataViews: Mark the new Templates pages as stable #57109
Conversation
I noticed that the existing page has a "Add template" button at the top, not sure if want to add that button first before merging this one. |
@@ -1,92 +1,407 @@ | |||
/** |
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.
This is just the old dataviews-template.js without any change.
Size Change: -67 B (0%) Total Size: 1.71 MB
ℹ️ View Unchanged
|
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.
Thanks Riad!
existing page has a "Add template" button at the top
We should probably add this button too and let's ship.
Flaky tests detected in 740d308. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/7223074758
|
Nice, that was fast. :) |
Related #55083
The idea here is to start getting some real usage feedback about the dataviews, to be completely ready when we reach the beta1 for the new WP release.
Testing instructions