imagefact

Thanks for posting this – and finding out where the problem is. I second it! Since this was marked as resolved (even though it isn’t) they might not see this note. You may need to post as a new item.

Ok, that code caused a lockout of the site. But this code works:

/* Change PublicPress notifications trigger */
/* Don't Trigger notification when trashing posts */
add_filter('pp_notification_status_change', 'ppchangetrigger');
function ppchangetrigger($new_status) {
if ($new_status === "trash") {
		return false;
	}
return true;
}

As for the other problem of triggering a notification on submitting a post. What I thought seemed to be true. On the page you sent:
publishpress/modules/notifications/notifications.php

On line 108 I changed the 10 to a 10000 and it then works:
add_action('transition_post_status', array($this, 'notification_status_change'), 10000, 3);

But of course, this is hardcoded and will get lost. So I have 2 questions for you:

1 – If there a way around this?
2 – Can you just permanently change this in your code on next update? Will it cause conflicts for you/others?

Thank you! So would it be like this?

/* Change PublicPress notifications trigger  		*/
/* Don't Trigger notification when trashing posts 	*/
add_filter('pp_notification_status_change', 'ppchangetrigger');
function ppchangetrigger($new_status, $old_status, $post, 10,3){
    if $new_status === "trash" {
    	return false;
    }
    return;
}

ALSO ANOTHER QUESTION:
This is sortof the opposite from above, where I WANT to trigger a notification on publish, but it’s not working:

I’m using the wp-types plugins and I have a front-end submit button (Cred-Submit) that pubishes posts from the front-end form. It doesn’t seem to be triggering your notification. One thought is that their priority is set later than yours.

1 – Where can I find the priority in your plugin to set it at late as possible to see if that fixes the problem.
2 – Any other thoughts on what could fix this?

I will also see if I can find any solution on their end, but thought I’d start with this.

Very late on this but… Try zapping it out with css. Find your id# or class and:

display:none;

Yes, saw those but the hooking wasn’t working. Hmmm… I’ll look into it more to see if I can come up with a more specific question then. Thanks for answering!

Fix causes other errors on my site….

Didn’t know about the other plugin – great to see.

A combo that allows internal links to look one way and external another would be terrific.

Yes, this helped. Thank you!

Would be great to have it in the real plugin. : )

I needed to add this in as well due to an error when trying to update passwords. THANK YOU!

Sed, I think it’s great what you’ve made – and it seems to have really powerful options, much more than other plugins probably have. So for those that want a really powerful option – I think this plugin is it… I think people just needed to know that the plugin shouldn’t be de-activated once it’s been in commission.

I’ll second that. Searching for an answer (I’ll ftp in the meantime).

Wow – should you put that on the plugin info? Seeing it not upgraded or compatible almost made me immediately more on… it’s good to know!

I’m truly impressed. I’m looking around at all sorts of multisite backup solutions and one of the features I’m seeking is resolved by you just this month! How wonderful an Author you are and how fortunate my timing!

If I do wind up using your plugin you’re the kind of author I would certainly donate to…

Not sure if this will help you or not, but if you haven’t seen it – try this:
https://yoast.com/gravity-forms-custom-post-types/

THIS IS AWESOME. I can’t thank you enough – well yes, I can, I’ll need to send a donation!

So I know for the future: Is this going to be integrated into a future release?

For those that don’t know what to do with this. Make it into a text file and put in a folder inside your plugins folder. I found an example explanation here:
https://www.doitwithwp.com/create-functions-plugin/