-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
1fc791a removes the slider from cbox-theme/infinity and moves it to a separate plugin.
Among various backward-compatibility concerns with the theme rewrite, this one (along with settings migration, which is already handled quite elegantly) is critical. Users who upgrade their theme should be able to migrate with minimal effort - ideally zero effort.
A couple of ideas for discussion:
- Modify commons-in-a-box to support the following workflow: the cbox-theme theme will require the infext-slider plugin, so that when activating cbox-theme, infext-slider will be automatically installed and activated. @r-a-y How much work would this be?
- Add infext-slider to the list of Recommended plugins, to be installed when commons-in-a-box is updated/installed.
- Do (1) for existing users of the slider (as determined by, say, querying for the CPT), and (2) for other installs.
- As part of the cbox-theme distribution build process, grab infext-slider from its separate repo. Put it somewhere into the theme itself, where it can be
included. (This allows development to be separate, but doesn't help with the issue of distributing a plugin inside of a theme.) - Do nothing automatic for existing users, but show them a notice that they should install infext-slider. Presumably this would go hand-in-hand with (2). (I strongly recommend that we don't do this, but just throwing it out there.)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels