r-a-y

@naomir – You mentioned that when you trash a forum post that it doesn’t remove the corresponding activity entry.

I cannot duplicate this issue on the latest versions of WordPress, BuddyPress and bbPress. I trashed a forum post from the group forum thread and the corresponding activity entry was removed from the group activity stream.

Are you doing anything different? Are you able to replicate your problem on a fresh install?

Thanks for the report.

A fix is available here – https://github.com/boonebgorges/buddypress-group-email-subscription/pull/192/files

Thanks for responding, David.

It would be great if you hosted your code on Github, Gitlab or whatever works for you, then we could send pull requests for your review.

I wanted to rewrite the way your plugin displays authorization screens with wp_die() instead of how it uses a variant of login_header() at the moment.

There are some other things as well, but we’ll leave that until it’s possible to send pull requests.

@tivus – Your fork looks cool as well, although there are some things I would also do differently as well.

The new version of the plugin does self-pings to initialize the email sending process asynchronously.

Read the 3rd point in this wiki article:
https://github.com/boonebgorges/buddypress-group-email-subscription/wiki/How-to-troubleshoot-missing-emails

You’ll need to figure out why self-pings are not working on your server. If self-pings are working on your server, perhaps it’s a problem with the way we are setting up the self-ping.

It sounds like your issue is you are using a 3rd-party email plugin.

3rd-party email plugins force the use of plain-text. When this happens, for the plain-text format, we convert HTML links over to legible links.

You can try the following code snippet to disable plain-text conversion for usage with your 3rd-party email plugin:
https://github.com/boonebgorges/buddypress-group-email-subscription/wiki#3-i-upgraded-to-v370-and-foreign-characters-look-incorrect-in-my-email-why

You’re using an older version of the codebase from GitHub.

Upgrade to the latest master version:
https://github.com/r-a-y/buddypress-followers/archive/master.zip

It sounds like you are using a 3rd-party email plugin.

If that is the case, please read the following:
https://github.com/boonebgorges/buddypress-group-email-subscription/wiki#3-i-upgraded-to-v370-and-foreign-characters-look-incorrect-in-my-email-why

Hi, thanks for the report.

Can you try reading this migration article to see if that helps or not?
https://github.com/boonebgorges/buddypress-group-email-subscription/wiki/Migrating-to-3.9.0

The latter part of that article explains that there should be an option on the admin settings page to start the migration process. If you are familiar with WP-CLI, there are also commands you can use to create the DB tables. This info is also available in the article linked above.

• This reply was modified 6 years ago by r-a-y.

Sorry for the late reply. This should be fixed in the development version:
https://github.com/r-a-y/buddypress-followers/archive/1.2.x.zip

Hi, the “New Topics” option requires installing the bbPress plugin.

Ensure that you have installed bbPress and let us know if the option shows up afterwards.

This is a BuddyPress problem.

Follow the first point I listed here to remove the ‘Reply-To’ address:
https://buddypress.org/support/topic/confirmation-emails-are-not-sent-for-new-users/#post-274356

You could potentially modify that snippet to set a different Reply-To address, but you’ll have to do so yourself.

@breity10 is using the plain-text email, which converts HTML tags to a Markdown-like syntax.

If you want to remove this functionality, use the snippet listed here:
https://github.com/boonebgorges/buddypress-group-email-subscription/wiki#3-i-upgraded-to-v370-and-foreign-characters-look-incorrect-in-my-email-why

—-

It might be a helpful tool for a new token like {{{ges.activity_id}}} to be passed through to this filter, so that the tokens can be rebuilt in a more dynamic fashion. @r-a-y do you have thoughts about this, or corrections to the above

Activity object is already passed in the email sending function:
https://github.com/boonebgorges/buddypress-group-email-subscription/blob/master/bp-activity-subscription-functions.php#L428

So you should be able to use the 'ass_send_email_args' filter to do whatever you need to do:
https://github.com/boonebgorges/buddypress-group-email-subscription/blob/master/bp-activity-subscription-functions.php#L509

Example:

add_filter( 'ass_send_email_args', function( $retval, $email_type ) {
	if ( 'bp-ges-single' === $email_type ) {
		// Do checks against the activity object and do whatever.
		$activity_obj = $retval['activity'];
	}

	return $retval;
}, 10, 2 );

@corgdesign, that’s also a neat way to fix this.

Better than my solution! ??

Hi,

Someone posted about this problem on the BuddyPress forums:
https://buddypress.org/support/topic/404-error-when-user-clicks-on-email-message-link-not-logged-in/

I’ve added a fix there but it relies on registering a subnav menu item. If you don’t want to register a subnav, you could also probably create your own redirector routine as well.

Thanks for the report. BuddyPress 3.0 broke this plugin due to removing support for the older, legacy forums.

Should be fixed here – https://github.com/r-a-y/buddypress-usernames-only/commit/2fda19686137ad7f61f633ccd5a7aec5b6b2937f

Other than that, it should be compatible with WP 4.9.6.

• This reply was modified 6 years, 10 months ago by r-a-y.