[plugin Better Notifications for WordPress] suggestion

  • Resolved rwatuny

    (@rwatuny)


    9 years, 3 months ago

    Hello,

    Nice plugin.

    If I may make a suggestion, in bnfw.php, just under
    add_action( 'draft_to_publish' , array( $this, 'publish_post' ) );
    you could add this line
    add_action( 'auto-draft_to_publish' , array( $this, 'publish_post' ) );

    It helps when a draft has been auto-saved by WP before having been saved.
    Also it makes this plugin work with the DJD Site Post front end editing plugin.

    Regards.

    https://wordpress.org/plugins/bnfw/

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Author bnfw

    (@voltronik)

    9 years, 3 months ago

    Hi rwatuny,
    Thanks for your message.

    I can certainly look to add it but will need to do some testing first to ensure it doesn’t push out lots of notifications.
    Can you elaborate on the compatibility issue you’re having with DJD Site Post?

    Thread Starter rwatuny

    (@rwatuny)

    9 years, 3 months ago

    Hello,

    Thank you for a very quick response.

    As you may know, the DJD Site Post plugin adds a (responsive) form to to the front-end of a WP site to write a post without having to go into the admin section.

    It uses the auto-draft status when first creating an empty post.
    So when publishing a post, it triggers the auto-draft_to_publish action.

    Since your plugin uses the draft_to_publish action hook, no notification is sent when publishing with DJD Site Post.

    Thank you for your time.

    Regards.

    Plugin Author bnfw

    (@voltronik)

    9 years, 3 months ago

    Hi rwatuny,

    Ah, I see! I wonder why they chose that over draft_to_publish?
    I’ll look to add it in in v1.1 after some testing.

    I’ll leave this topic open for now.

    Thread Starter rwatuny

    (@rwatuny)

    9 years, 3 months ago

    Hello,

    I wonder as well…

    Thank you.

    Plugin Author bnfw

    (@voltronik)

    9 years, 3 months ago

    Hi,

    I’ve now looked into this and think it’s best that the other plugin author change the hook they’re using. I think they way it works in BNFW is best for this particular plugin. Sorry I can’t be of anymore help here.

    Do feel free to get in touch with the plugin author in the hope that hey might change it.

    Thanks,
    Jack

    Thread Starter rwatuny

    (@rwatuny)

    9 years, 3 months ago

    Hello,

    I will contact the other plugin’s author.

    By curiosity, what are the reasons for your decision ?

    Regards.

    Plugin Author bnfw

    (@voltronik)

    9 years, 3 months ago

    We tested this quite a bit before whilst developing the beta and found that it triggered notifications during auto-saves which isn’t how you want notifications to be delivered – you only really want them when they’re say, published or updated.

    Thread Starter rwatuny

    (@rwatuny)

    9 years, 3 months ago

    Hello,

    I see. You are absolutely right.

    Thank you again.

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘[plugin Better Notifications for WordPress] suggestion’ is closed to new replies.