wpsharks / comment-mail

A WordPress plugin enabling email subscriptions for comments.
http://comment-mail.com
GNU General Public License v3.0
8 stars 3 forks source link

Comments submitted via WP REST API are not auto-subscribed #321

Open raamdev opened 7 years ago

raamdev commented 7 years ago

We have a report on WordPress.org that comments submitted via the WP REST API are not auto-subscribed when Comment Mail → Config. Options → Auto-Subscribe Settings are enabled.

I've posted a comment on the thread asking for more details for reproducing this.

This needs to be tested to confirm this is a bug.

renzms commented 7 years ago

@raamdev

Was able to reproduce customer issue

However, I noticed commenting via REST API did not provide any email alerts at all, even for the default WordPress Admin alerts that a new comment was added. This may be a limitation for WordPress REST API v2. There were alerts for all comments including those made via REST API in the Jetpack notifications and WordPress.com Desktop App though.

Tested with

WordPress Version: 4.6.1 Current WordPress Theme: Twenty Sixteen version 1.3 Theme Author: the WordPress team - https://wordpress.org/ Theme URI: https://wordpress.org/themes/twentysixteen/ Active Plugins: Comment Mail Version 161129 | WP REST API Version 2.0-beta15 | Jetpack

Using: Advanced REST Client Chrome App WP REST API Schema for comments

Step Taken

raamdev commented 7 years ago

@renzms Thank you.

@jaswsinc This could use a review and an estimate when you get a moment.

jaswrks commented 7 years ago

Estimating this at one day. Based on Renz's testing, it sounds like we are missing a hook or validation somewhere that just needs to be found and connected properly. Most of this will be about me setting up a testbed myself and then working to locate the proper hooks.