Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 399
  • Merge Requests 52
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !613

Open
Opened 4 weeks ago by Ben Hayward@benhayward.ben
Report abuse

WIP: [Sprint/QuietQuail](fix): Blog & media edit updated to new nsfw system #1936

Closes #1936 and #2267

Depends on engine!384

Summary

This changes first part encompasses a fix to the NSFW system, wherein even though the selector shows as blue, it does not respect the values in it without clicking it, so a user may believe they are posting something as NSFW, but in actuality the post would be untagged.

The other change, moves the future blogs to the mature blog system, to the NSFW blog system. When creating a new blog, values must also be respected as in the first part of this upgrade, however, when editing, existing values must be shown.

Testing

Blog

Example testing video - https://streamable.com/ps39o

  1. Post NSFW Post, give it a few random reasons, remember them.
  2. Post a blog - do not touch the selector
  3. Edit blog, check the selector - it should be the same as your last post.
  4. Change the NSFW tags
  5. Edit again and ensure the change stuck
  6. Check the changes are respected on the channel sidebar
  1. Post a non-NSFW blog post
  2. Edit it to be NSFW, enter edit mode again to check it 3. worked
  3. Make a normal post with different tags.
  4. Edit blog and check it ignores those tags, and has the correct ones for the blog.
  1. In one of the NSFW blogs activity posts on the feed, mark it as non-nsfw
  2. Check change propegates by clicking the blog, editing, checking nothing is checked
  3. Ensure the sidebar of the blog does not have the explicit overlay now.
  1. Edit it back to be NSFW in the activity.
  2. Check the change propegated in the same fashion.
  3. Ensure blogs in sidebar are flagged appropriately
Poster

Example testing video - https://streamable.com/3r8d2

  1. Make a non-nsfw post.
  2. Edit to be nsfw.
  3. Edit again to make sure the tags stuck.
  4. Remove NSFW from the activity, check the activity is no longer nsfw.
  5. Make a new NSFW post, remember the nsfw tags you placed on it.
  6. Nav to another page like wallet and back.
  7. Post, it should be NSFW.
  8. Make another post with a different config to change the nsfw selectors defaults.
  9. On the post in step 7, edit and check the nsfw values are correct.
Media

Example testing video - https://streamable.com/qokw4

  1. Post NSFW media (remember which nsfw reasons you selected).
  2. Make another random post with another configuration of NSFW reasons.
  3. Check the activity, edit it, it should show the reasons correctly, and not show the values of your last post.
  4. Edit it to not be NSFW, this should be respected
  5. Edit it back to being NSFW.
  1. Go to the media page for the activity (if you are seeing media modals, just reload the url in the url box).
  2. NSFW values should be respected (check via edit)
  3. Edit away the changes
  4. Check changes apply to the activity on the newsfeed.
  5. Go back onto the media page.
  6. Make it NSFW again.
  7. Check changes propagate.
Admin

Example testing video - https://streamable.com/v8hye

  1. Watch XHRs
  2. Visit channel
  3. Ensure no spam calls are made to moderation endpoints
  1. Ensure an admin can set a post on another user as NSFW
  2. Ensure an NSFW post shows the correct reasons initially, regardless of what the current state of the admins NSFW post box is.
Edited 20 hours ago by Ben Hayward
Request to merge fix/blog-media-nsfw-update-1936 into master
The source branch is 11 commits behind the target branch
Open in Web IDE
Pipeline #97599328 running for 31888535 on fix/blog-media-nsfw-update-1936
              Requires 3 more approvals from Devs, Deployers, and QA.
              Emiliano Balbuena
              Emiliano Balbuena
              Mark Harding
              Mark Harding
              Brian Hatchet
              Brian Hatchet
              Xander Miller
              Xander Miller
              This is a Work in Progress

              Closes #1936 and #2267

              Deletes source branch

              You can merge this merge request manually using the
              • Discussion 1
              • Commits 11
              • Pipelines 6
              • Changes 11
              • Ben Hayward @benhayward.ben added MR::Awaiting Review Squad::Yellow scoped labels 4 weeks ago

                added scoped labels

              • Ben Hayward @benhayward.ben changed the description 4 weeks ago

                changed the description

              • Ben Hayward @benhayward.ben marked as a Work In Progress 4 weeks ago

                marked as a Work In Progress

              • Ben Hayward @benhayward.ben added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 4 weeks ago

                added scoped label and automatically removed label

              • Ben Hayward @benhayward.ben mentioned in merge request engine!384 4 weeks ago

                mentioned in merge request engine!384

              • Ben Hayward @benhayward.ben changed the description 4 weeks ago

                changed the description

              • Ben Hayward @benhayward.ben added 2 commits 4 weeks ago

                added 2 commits

                • b3bf6a83 - Updated test
                • be7434a2 - Updated E2E tests to use newer nsfw selector

                Compare with previous version

              • Ben Hayward @benhayward.ben added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 1 week ago

                added scoped label and automatically removed label

              • Ben Hayward @benhayward.ben unmarked as a Work In Progress 1 week ago

                unmarked as a Work In Progress

              • Ben Hayward @benhayward.ben added 90 commits 2 days ago

                added 90 commits

                • be7434a2...a0d1737e - 87 commits from branch master
                • 7cb030ad - Merge branch 'master' of gitlab.com:minds/front into fix/blog-media-nsfw-update-1936
                • aaf95d86 - Merge branch 'master' of gitlab.com:minds/front into fix/blog-media-nsfw-update-1936
                • 76664e6c - Merge branch 'master' of gitlab.com:minds/front into fix/blog-media-nsfw-update-1936

                Compare with previous version

                Toggle commit list
              • Ben Hayward @benhayward.ben added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 1 day ago

                added scoped label and automatically removed label

              • Ben Hayward @benhayward.ben marked as a Work In Progress 21 hours ago

                marked as a Work In Progress

              • Ben Hayward @benhayward.ben changed the description 21 hours ago

                changed the description

              • Ben Hayward @benhayward.ben changed the description 21 hours ago

                changed the description

              • Ben Hayward @benhayward.ben added 2 commits 20 hours ago

                added 2 commits

                • f91890c3 - Blogs working correctly
                • 36f95435 - spec test fix

                Compare with previous version

              • Ben Hayward @benhayward.ben changed the description 20 hours ago

                changed the description

              • Ben Hayward
                Ben Hayward @benhayward.ben · 1 hour ago
                Developer

                This definitely needs testing with live data, so we can see how old blogs marked as mature will react to the changes.

              • Ben Hayward @benhayward.ben added 1 commit 48 minutes ago

                added 1 commit

                • 31888535 - Removed console logs

                Compare with previous version

              Please register or sign in to reply
              0 Assignees
              None
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              2
              Labels
              MR::Requires Changes Squad::Yellow
              Lock merge request
              Unlocked
              10
              10 participants
              user avatar
              Emiliano Balbuena
              user avatar
              Mark Harding
              user avatar
              Guy Thouret
              user avatar
              Brian Hatchet
              user avatar
              Olivia Madrid
              user avatar
              Martin Santangelo
              user avatar
              Marcelo Rivera
              Reference: minds/front!613