Skip to content
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

Emails sent right away, and no documentation how to use pmproeewe_test #20

Open
amityweb opened this issue Dec 24, 2017 · 2 comments
Open

Comments

@amityweb
Copy link

I enabled this and all our members expring within 60 day got a notice. I did not want 60 days, only 30. Everyone is kicking up a fuss now.

Out of the box this is a dangerous plugin. I do not think any periods should be enabled out of the box. I think it should be more clear the plugin will send reminders right away.

There is is also no documentation on how to use pmproeewe_test. Ideally when I enabled this I should be able to set the expire period and do a pmproeewe_test, but the period is already set, and pmproeewe_test is off, so emails go right away. This is not clear, I expected to set the periods in the admin settings when I enabled it, and not expecting it in the code itself.

@amityweb
Copy link
Author

amityweb commented Dec 24, 2017

Also not much documentation about other things... I notice this in the code //first, disable the default email, so does that mean the 7 days notice wont be sent? I think this plugin needs more documentation as people like me will just install it and not realise it will start working right away, and expect some sort of Admin interface to set and enable the reminders. Its no problem for me not having that interface, but it was not expected.

@ideadude
Copy link
Member

ideadude commented Sep 4, 2018

This is good feedback. I hope we can work to improve this plugin, both to streamline the setup for non devs, and improve the documentation to make it more clear what happens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants