Comment 5 for bug 1661688

Revision history for this message
Mike Rylander (mrylander) wrote :

Thanks, Dan! This looks like a good improvement, and doesn't forestall the "friends" variant.

I have one nit to pick, though, that may require some input. The global default for the new checkbox is to be checked, which is a behavioral change. As a rule we try not to push a configurable behavior change at upgrades, so I think this should probably not insert the setting at the top of the org tree as it does currently, and rely on the release notes to inform ILS admins about the new feature and how to configure it.

Thoughts?