-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat(gsoc'24): report as spam and moderation review page #25
Conversation
Could you resolve the conflicts ? Also not all of these messages are spam. Make spam as a category and rename the text as "Report post". Also don't push mark as solution to the dropdown, it's visibility will decrease. |
Yeah agreed with your point, but if we add this button in the forum post's footer, it will make post more crowdy with this action button. Since all action regarding the forum post is now grouped using this dropdown, it just matter of time user will adopt to this feature as they gets used to this forum card layout. wdyt? |
…r` gem - these filters are applied on forum_thread's title, forum_posts's body - error messages are now improves with proper altert message styling using bootstrap's alter class
@tachyons made that |
a595756
to
3a76984
Compare
@tachyons I've added new tests for new functionality which I added. can you please re-review this PR |
Fixes #17
This PR's covers following functionality
threads/spam_reports
Note: For client side interaction like dropdown toggle, report spam modal form, javascript was needed so I've used hotwire stimulus controllers for this. and currently build files of these javascript assets are not pushed but to use this rails engine in production, we have to push these javascripts assets with the rails engine.
Screenshots: