CUNY Academic Commons

On the Commons currently, we do not use AJAX for the follow sites button.

Most importantly, the follow AJAX and JS functionality is currently only coded for the members component and I believe we have disabled the follow members portion on the Commons, so the follow AJAX handlers would also not be registered causing the 400 error.

I can make some adjustments to allow AJAX to work for the follow sites button as well.

Most importantly, the follow AJAX and JS functionality is currently only coded for the members component and I believe we have disabled the follow members portion on the Commons, so the follow AJAX handlers would also not be registered causing the 400 error.

This all explains why it didn't work :)

I don't think AJAX is absolutely necessary, but if we don't have AJAX (and I guess even if we do, as fallback) then the redirect behavior should be more logical. The Follow buttons I have in place currently work, but you end up on the home page.

Thanks for having a look!

I don't think AJAX is absolutely necessary, but if we don't have AJAX (and I guess even if we do, as fallback) then the redirect behavior should be more logical. The Follow buttons I have in place currently work, but you end up on the home page.

For now, I've opted to fix the Follow button on the Sites Directory page without AJAX support - https://github.com/cuny-academic-commons/cac/commit/7f13480c8ee57616eee0a68953a5cb28b4755e9b.

Refactoring BP Follow's JS to add in Sites support would require quite a few changes since the nonces to validate the AJAX action are different for the components. It would probably be easier to write a custom AJAX follow sites handler specifically for bp-nelo. More to come.

Nice! I've tested and this works well for me.