Track Akismet spam checks, reflect on comment addition log #472 by shadyvb · Pull Request #587 · xwp/stream
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 115
Merged
Track Akismet spam checks, reflect on comment addition log #472#587
frankiejarrett merged 3 commits intodevelopfrom
Track Akismet spam checks, reflect on comment addition log #472#587
frankiejarrett merged 3 commits intodevelopfrom
Conversation
Copy link Copy Markdown
Contributor
shadyvb
commented
Jun 29, 2014
shadyvb
commented
Fixes #472 by properly detecting Akismet response on new comments, and reflecting on new comment log comment_type added in #558. So users now should be able to filter auto-spammed comment easily.
@fjarrett @lukecarbis Please review/test.
/cc @krogsgard Can you checkout this branch and confirm this solves your issue on #472 ?
connectors/comments.php Outdated
Copy link Copy Markdown
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shadyvb Needs to be Yoda style
connectors/comments.php Outdated
Copy link Copy Markdown
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shadyvb I think we should also add some Steam meta here as well as indicate in the summary that Akismet had involvement with the action.
Copy link Copy Markdown
Contributor
frankiejarrett
commented
Jul 7, 2014
frankiejarrett commented
Jul 7, 2014I've got this branch installed on my personal site (which gets a lot of spam). I will see how it goes over the next several days and report back here with the results.
Copy link Copy Markdown
Contributor
frankiejarrett
commented
Jul 8, 2014
frankiejarrett commented
Jul 8, 2014This is working great! Ready for merge.
frankiejarrett added a commit that referenced this pull request
Jul 8, 2014Track Akismet spam checks, reflect on comment addition log #472
frankiejarrett
merged commit
5abc963
into
develop
frankiejarrett
deleted the
issue-472
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants