cuny-academic-commons / commons-in-a-box

Commons In A Box - A suite of community and collaboration tools for WordPress, designed especially for academic communities
http://commonsinabox.org
72 stars 14 forks source link

Port Template Chooser fixes from City Tech OpenLab #457

Closed bree-z closed 9 months ago

bree-z commented 1 year ago

This ticket is for Boone to review whether the fixes we made to the template chooser for the City Tech OpenLab in 3227 should be ported to CBOX OpenLab.

boonebgorges commented 10 months ago

I've taken a look at the Redmine ticket and at what's currently in CBOX-OL. It doesn't appear to me that any of the issues that were present on the OpenLab are occurring in CBOX-OL, because they all came from the compatibility layer that was required to use the template picker on the OpenLab. @bree-z Perhaps you could review yourself before closing this ticket.

bree-z commented 10 months ago

Thanks Boone. I'm not sure if I'm seeing the same thing on openlabdev.commonsinabox.org as we did in 3227, but I tried the following two things:

  1. Set the default template for projects, then disassociate that template from projects.
  1. Set the default template for certain group types, then unpublish the default template.
boonebgorges commented 10 months ago

Thanks for spelling this out. I now recall some of the more unpleasant details of the original report. I've ported some of the logic introduced on the OpenLab.

bree-z commented 9 months ago

Thanks Boone! I wasn't able to reproduce the issues described above.

I'm going to do a bit more general template testing, so I won't close this out yet.

bree-z commented 9 months ago

I think we can close this, thanks!