CUNY Academic Commons

Meeting notes, specific to this ticket:

- Track plugins from a package that might have been deactivated manually (storage method should use `get_option()` instead of `bp_blogs_get_blogmeta()` here unless we want to track deactivations across the network).
- Inform admins of new plugins to a package (on the "Plugins > Packages" page, dashboard metabox and potentially elsewhere)

- Track plugins from a package that might have been deactivated manually (storage method should use `get_option()` instead of `bp_blogs_get_blogmeta()` here unless we want to track deactivations across the network).

Or do it dynamically:

$this_site_packages = bp_blogs_get_blogmeta( $blog_id, 'plugin_package', false );
foreach ( $this_site_packages as $package_id ) {
  $package_plugins = cac_get_plugin_package( $package_id );
  $active_plugins = get_option( 'active_plugins' );
  $diff = array_diff( $package_plugins, $active_plugins );
}

Hi Ray - This is looking great, really above and beyond. Thanks!

Definitely put the plugin-installation question on the backburner. On the Commons, we'll ensure that plugins in packages are installed.

I agree -- this looks awesome, Ray!

Chris and Sonja-- please review plugin-list-table.png and see whether you think "Teaching | Multimedia" (even if they are clickable and go to anchored links on the Plugin Packages page) will be intuitive to people

Suggestion during the meeting:
add a line break after "View details"
then say "Plugin Packages: Teaching | Multimedia"

Raymond Hoh wrote:

I've gone ahead and moved the Plugin Packages tags onto its own line.

See attached screenshot:

Let me know if any changes are required.

Ray, dropping to second line and adding "Plugin packages:" makes this much clearer! Great job!

Raymond Hoh wrote:

I've added the plugin to 1.14.x branch here - https://github.com/cuny-academic-commons/cac/commit/b05ef038611fbf13503db46cdbc13032375dc0c2

Here are some updated screenshots of some new functionality:

Dashboard widget

Part of the spec wanted some form of reporting in the dashboard. With that in mind, I've added a "Plugin Packages" dashboard widget. This widget only shows up if the site has previously activated a plugin package.

It reuses the package tags as shown before.

Also, if some plugins are inactive in a package, a small note is added with a brief description to tell the admin to click on the package tag for more info.

I wonder if the notice "Some plugins are inactive..." on the dashboard widget will grab admin's attention the way it is implemented now. I think we should make it look more like an alert than simple text. Maybe repeat the circular "i" icon for consistency?!

Ray, in today's meetings, the team finalized the spec for which plugins will be included. We'd like the following changes:

- To Teaching, please add:
- https://wordpress.org/plugins/hypothesis/
- https://wordpress.org/plugins/wp-accessibility/

- To Digital, please add:
- https://wordpress.org/plugins/knight-lab-timelinejs/
- https://wordpress.org/plugins/google-docs-shortcode

Thanks!

Raymond Hoh wrote:

Thanks for the feedback, Sonja. I've added the "i" info icon to the dashboard widget.

Thanks Ray, that draws enough attention to read it!!

I just noticed that the Plug In Package picker has Lorem Ipsum text in the description field immediately below the header.

I just noticed that the Plug In Package picker has Lorem Ipsum text in the description field immediately below the header.

Oof! Good catch, Steve.

I've just pushed a commit to remove the placeholder description. If we want to put in some helper text or with a link to some documentation, I can also add that as well.