-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Related to checkbox #523
Comments
Hi, We'll get that fixed. In the meantime can you try this:
|
Can you please tell me about how much time it would take to fix this issue? I'm totally rely on this plugin. If you'll tell me the time or date then I would check plugin back on that date. |
Did you try the method I mentioned in the previous message. Basically what happens is that the option doesn't update when "Advanced Subscription" is disabled. But you can enable it, change the option and disable it, the value will remain. It's not a fix, but it's a temporary workaround that has the same result as the fix will have. As for when the update will be release, it depends on the other fixes and improvements that are in the next update. And there's 20 other plugins/themes that also have ongoing maintenance and updates. Which makes things had to predict and plan for. Might be tomorrow but might be next week. Sorry, can't say for sure. |
Yes, I followed these steps:
But when I click save changes two times after switching “Advanced Subscription” to “No” then it again switch to "All new comments" I should not click save changes two times. And yes it is working. One more thing I need to ask: When WordPress is sending email to users for new reply added to comment then it is landing in their SPAM folder. Is it because I have no SMTP enabled? Can you please tell how can I connect this plugin with SMTP? |
I have a PR for this #522 |
When I set Default Checkbox Value to "Replies to this comment" and hit the Save Changes button then Default Checkbox Value automatically updated to "All new comments". This is happening again and again.
See this video to better understand the issue: https://we.tl/t-m8MuXR47LZ
The text was updated successfully, but these errors were encountered: