cuny-academic-commons / cbox-theme

Default theme for Commons In A Box
GNU General Public License v2.0
20 stars 15 forks source link

BuddyPress 3.0 compatibility #260

Closed r-a-y closed 6 years ago

r-a-y commented 6 years ago

Update: This issue was originally created to cover activation template changes, but now encompasses all issues with BuddyPress 3.0.


In BuddyPress 3.0, the activation template has changed.

Previously, simply loading a URL like /activate/12345 would activate the account with key 12345.

BP 3.0 no longer does this. For CBOX theme, we'll need to mirror the following changeset in order to auto-populate the activation key field: https://buddypress.trac.wordpress.org/changeset/11766/trunk/src/bp-templates/bp-legacy/buddypress/members/activate.php

r-a-y commented 6 years ago

@boonebgorges - I've added the activation template change to master branch.

One thing I thought of is do we have to worry about sites using < BP 3.0? If so, we'll have to change up the template to add some conditionals. I guess this applies more for those cloning bp-default.

boonebgorges commented 6 years ago

Once we bump the minimum BP requirement in CBOX, it should be updated at the same time as cbox-theme. So I don't think we need to account for older BP. Does that sound right?

r-a-y commented 6 years ago

I was thinking about those that clone and pull cbox-theme (or bp-default) without updating BuddyPress to v3.0 immediately.

In those instances, the activation page would throw a fatal error due to the use of a BP 3.0 function. Am I overthinking things?

boonebgorges commented 6 years ago

Ah, I see. Yeah, I think that's overthinking. Anyone doing weird stuff should know enough to test :)

r-a-y commented 6 years ago

Gotcha. Thanks for the sanity check!