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
Hide moderated content from notifications #12829
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've just checked it out locally and it works as expected. Just a couple details to change and we're good to go
decidim-accountability/spec/events/decidim/accountability/proposal_linked_event_spec.rb
Outdated
Show resolved
Hide resolved
decidim-accountability/spec/events/decidim/accountability/result_progress_updated_event_spec.rb
Outdated
Show resolved
Hide resolved
decidim-comments/lib/decidim/comments/test/shared_examples/comment_event.rb
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to confirm it, but we can't show the date when it was moderated so it's consistent with the message in the comment page, right? (i.e. "Comment moderated on 23/05/2024 11:05")
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As i have previously stated, we cannot display that information, or at this moment i do not see how to do it.
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
🎩 What? Why?
This PR attempts to hide the moderated content from the notifications block. I have noticed that the moderated content is still being available in the notifications even though the content itself has been moderated. I have noticed that a while ago in MetaDecidim.
This PR adds a simple "Content moderated" string instead of actual notification, just to preserve the pagination integrity. It does not add any "Content moderated on %{date}" as we have on comments, because we do not have a very good grasp of the resource that is being hidden. Sometimes, the hidden content can be a comment, but sometimes it could be the resource commentable.
📌 Related Issues
Link your PR to an issue
Testing
📷 Screenshots
Please add screenshots of the changes you are proposing
A comment being moderated
A resource being moderated ( that also have comments associated: 1 meeting + 4 comments associated )