Fix comment videos front#6069

Ticket(s) / Related Merge Requests

front#6069

Summary of Changes

Fix for videos not playing in the comments. Comment videos have the access ID of the USER who owns the comment - this causes a failure in the adjusted ACL hook, which does not run an ACL check on the user.

Testing Considerations

Videos do not work correctly in sandboxes, thus actual verification of seeing a video will have to be done on staging.

Sandbox

• Check this endpoint returns data when logged out https://fix-comment-images-f6069.oke.minds.io/api/v2/media/video/1538507299256012811
• Check this endpoint returns data when logged in https://fix-comment-images-f6069.oke.minds.io/api/v2/media/video/1538507299256012811
• If you want to test your own video, get the above URL from the network requests on your video.
• Images in comments (owner / non owner / logged-out) https://fix-comment-images-f6069.oke.minds.io/newsfeed/1539220932302737412
• Images in posts (owner / non owner / logged-out) https://fix-comment-images-f6069.oke.minds.io/newsfeed/1539220932302737412
• Videos in posts (get /media/video endpoint response and check it returns something) (owner / non owner / logged-out)
• Videos in groups (get /media/video endpoint response and check it returns something) (owner / non owner / logged-out) https://fix-comment-images-f6069.oke.minds.io/newsfeed/1539221277141635090
• Images in groups (owner / non owner / logged-out) https://fix-comment-images-f6069.oke.minds.io/newsfeed/1539221920375902223
• Videos in comments in a group (get /media/video endpoint response and check it returns something) (owner / non owner / logged-out) https://fix-comment-images-f6069.oke.minds.io/newsfeed/1539221920375902223
• Images in comments in a group (owner / non owner / logged-out) https://fix-comment-images-f6069.oke.minds.io/newsfeed/1539221920375902223

Additional staging testing

Make a comment with a video.

• You can view it from the owner.
• You can view it from a logged-out state.
• You can view it from another user.

Deployment Considerations

Standard deployment.

Regression Scope

Comment videos.

Platform Affected (web, mobile, etc)

Engine.

Developer Testing Completed

Manual.

Screenshots / Screen Recording

simplescreenrecorder-2023-08-16_13.00.16

Does this impact

• Localization
• Dark/light mode
• Guest mode

Definition of Done Checklist

• The Acceptance Criteria has been met
• Code is tested: Testing includes unit/spec, E2E/automated and manual testing
• Merge requests description has been filled out