Skip to:
Content
Pages
Categories
Search
Top
Bottom

Re: Invite Anyone – invite by email too – please help test


Boone Gorges
Keymaster

@boonebgorges

Anton and hnla – thank you so so much for your very helpful comments. (I take it that most of the big functionality is working properly, since these are relatively small issues – that’s good news!)

Anton – Great point. I had originally not hardcoded the register link, but then had to hardcode it for some testing purposes and forgot to change it back. The latest trunk version creates the accept_link with site_url( BP_REGISTER_SLUG ), which is the same code used in BP core. Please let me know if it works for you.

hnla – Very sorry about my American date formatting :) I have changed it so that the format matches the setting in Dashboard > Settings > General. As for markup in the welcome message: I didn’t manually strip tags because I’m using $wpdb->prepare to write to the database, which means that all tags are escaped and therefore presumably not harmful. But I thought about it, and I think you’re right that it’s a good idea not to allow HTML in that box – it tends not to work well in email. The latest trunk version strips tags before sending the email/saving it to the db.

Finally, hnla, I’m guessing that your problem with /register/ redirects may have had something to do with one of the following:

– Anton’s problem (/register/ was hard coded into the accept_link)

– You are clicking on the link while logged in. BP redirects from the register page for logged in users. Log out, or try another browser.

– You’ve disabled registrations. In WPMU, look at Site Admin > Options; in WP I think it’s Settings > General > Anyone can register.

Thanks again for your help. I should be able to put a stable tag on it in the next day or so thanks to your feedback.

Skip to toolbar