-
Notifications
You must be signed in to change notification settings - Fork 28
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
Solved plugin problems? #77
Comments
@jeremycherfas Just tried all what you said, and it doesn't work for me. Nothing is saved in the |
I'm afraid I cannot help you. But let me test the comment function on my own site. I get so few comments that I can never be sure that it still works after upgrading Grav. Later: you're right, I am not receiving comments from my site and nothing is being saved. I need to check first that email sending works, and will then look elsewhere. Email works fine. @Romarain have you raised the question in Discord? |
@jeremycherfas Yes, I talked about that. But you know, for third-party plugins, people tend to ignore requests that seem more a consequence of our own config that of the plugin itself. |
Not sure what you mean about 3rd party. This plugin is made by the Grav team. |
I think I may have solved the problem of the plugin not working.
Line 192 originally read
$post = isset($_POST['data']) ? $_POST['data'] : [];
However,
$_POST
as received from the form did not contain any elementdata
. I replaced that line with the admittedly simplerThe plugin now works in my installation. I didn't actually fork the plugin, and so am not sure how to file a PR.
I also reversed
email
andaddComment
in the process section ofcomments.yaml
to avoid the problem of receiving an email even though the comment was not saved.This fix works even without
language
being set.The text was updated successfully, but these errors were encountered: