antispam-bee
antispam-bee copied to clipboard
Add report spam link
Add report spam action link to spam list (#73)
After the discussion: This is a nice feature. Thank you @Zodiac1978 for introducing it :heart:
Imo good to merge.
Maybe this is important to know. This PR adds the report link just to spam links. This is because of the already existing code.
Should we add this report link to all comments?
@websupporter I don't know why Travis is still using the old behat test. Even if I restart the job it is not using the changed test and therefore fails because of the changed string.
You would need to merge the master I think. But currently master seems to fail for coding standards :smile:
Imo its good to merge from the test side, as we know it just fails for the outdated test setup.
Ah sorry, I see it now I think. @schlessera did the first commit into an old branch (d53187d244ba28b7da2cb5f819576b416a8e345e) and than it was merged it into this one here (dd4ab8655bbb84a3f41a824e42be7c1bbe58375c). This branch here has the updated tests, this is why the first test fails while the second works imo.
Due to missing feedback I have punted this PR to milestone 2.10. :(
Without clear information about the privacy implications I don't see a possible path forward here.
Additionally I will close the form and delete the excel sheet, because I am the owner of it to protect me.
If anyone is seeing a possible solution to the whole problem, please add your comment. Otherwise I don't see another way as to close the hole issue/PR and remove any report mechanism due to unknown law implications.
- [ ] Add modal before sending data
- [ ] Use own form for storing the data
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
SonarCloud Quality Gate failed.
5 Bugs
0 Vulnerabilities
0 Security Hotspots
5 Code Smells
No Coverage information
0.0% Duplication
SonarCloud Quality Gate failed.
2 Bugs
0 Vulnerabilities
0 Security Hotspots
10 Code Smells
No Coverage information
0.0% Duplication
SonarCloud Quality Gate failed.
2 Bugs
0 Vulnerabilities
0 Security Hotspots
11 Code Smells
No Coverage information
0.0% Duplication