Trigger on PUBLISH not working

Hi, are you sure? How do you test the webhook?

https://drive.google.com/open?id=1v7mGLA9_XH_MMUBBieqVRwgUxEl7hUZl

Update also does not work https://drive.google.com/open?id=139ZAMT-Exf5K1BUltlaAo6b0vz9BzcVw

More information…..I tried using POsts instead of a custom post type
Update and Publish worked, and so did Draft but it posted 2x instead of just once

So perhaps its an issue with custom post types and Publish and Update

[ Please do not bump. ]

It could be interfering with other plugins installed. The multiple notifications were a problem since the beginning, but I found out the current configuration is the most robust.

None the less I’ll take another look at it.

Im using the current config

I have to test it through with Pods as I think it’s changing the default behavior of post publishing/updating.

Here to help, if you find anything with Pods. What action are you hooking into for the notifications to go out?

Hi Scott, please see the classes: https://github.com/BracketSpace/Notification/tree/develop/class/Defaults/Trigger/Post

@sc0ttkclark Yes! Love it when the big boys get involved! Scott I am using this to send several PODS metadata into a CRM. Whenever the POD is published, Drafted, or updated it should trigger this gentleman’s webhook and update the CRM. Its not triggering in some cases, it appears as though shortcodes are not executing, and its not sending metadata in the right format in some cases. I will record some screen shots and post for everyone. This plugin has great middleware potential and is a tighter integration than Zapier and without relying in a 3rd party.

Just tested it with a POD and it worked WITHOUT using shortcodes, and when I “Drafted” it the webhook ran 2x

Are you talking about our Pods forms/shortcodes themselves? Or how are you doing the inserting?

Sorry, let me explain better.
1) I have content that I wrote inside WordPress and customers in a CRM
2) The content is stored as 1 of 9 post types, and may be associated with any of 4 collections. Pods is behind the majority of it.
3) I keep the metadata related to my content within my CRM so that it can run logic and workflows based off of that.
4) Whenever a specific Post Types change, I want that change reflected in my CRM. This plugin is updating the content metadata in my CRM whenevr it changes

I’m not seeing any cases yet in the Notification plugin code where it would conflict with anything Pods does normally.

Pods does not call an independent wp_insert_post or wp_update_post except in the case when the API is used to save through. So normal post edit screen saves will still happen normally, we do our own saving hook into wp_insert_post to save any custom fields on the page but that shouldn’t be causing the double notification or no notification at all.

The only thing I saw was the draft trigger, where it does not check whether the previous status was previously set to draft already before running, which might cause duplicate draft triggers possibly? https://github.com/BracketSpace/Notification/blob/develop/class/Defaults/Trigger/Post/PostDrafted.php#L58

any progress on this?