-
Notifications
You must be signed in to change notification settings - Fork 11
Allow a snapshot to be scheduled #15
Comments
See also the ability to split up a snapshot into multiple: #16 |
@valendesigns here is the code from the transactions PR for handling the publishing of a transaction post: https://github.com/xwp/wordpress-develop/blob/97fd5019c488a0713d34b517bdbff67c62c48a5d/src/wp-includes/theme.php#L2071-L2120 For snapshots I suppose this logic would have to be short-circuited if |
There may need to be a “Snapshots” Customizer section that contains the fields for the future published date. This section could also contain a UI for selecting another snapshot to load, and also to provide a name for the snapshot (commit message), as in #19. See also the (defunct) Settings Revisions plugin: Nevertheless, I think it would be ideal if the controls for managing the snapshot remained in the header bar since they should be more meta. |
The issue I see with using the header bar is that switching could be problematic in the small amount of real estate we have up there. I guess we could have an edit button that opens a dialog which allows naming and switching? Or using a key event like control+click or similar. |
Note regarding calling |
Actually, I'm wrong about having to grant all caps during the save process. All we have to do is grant a cap that everyone has to all of the settings, and all of the settings will then be able to be saved. For example: foreach ( $settings as $setting ) {
$setting->capability = 'exist';
}
foreach ( $settings as $setting ) {
$setting->save();
} I loop over the settings first to set the |
This would implement https://core.trac.wordpress.org/ticket/28721
A snapshot should be scheduleable from the the Customizer via something like a “Schedule” button along with a timestamp for when it should apply.
If there is also an edit post screen for viewing a snapshot in the admin ( #18), the snapshot should also be able to be scheduled using the traditional post scheduling functionality.
The text was updated successfully, but these errors were encountered: