Skip to content

[engine] Blog images #2601

Ben requested to merge fix/blog-images-2601 into master

Ticket(s) / Related Merge Requests

Summary of Changes

Fixes an issue where-in blog images do not appear in blogs for users who are NOT the owner of a blog.

Testing Considerations

  • Post a blog with images - test when loading the blog from an incognito tab you can see the images.
  • Edit the blog and republish and re-check the above.
  • Retest in staging.
  • Note testing old blogs WILL NOT WORK ON SANDBOX as the images will be hosted cross-domain and they will be hosted on old, dead sandboxes. Even creating one on a currently operational sandbox it will fail the cross-domain checks (correctly). This will have to be tested on staging.

Quick test:

Test this blog, logged into the owner, logged out and logged into another user: https://fix-blog-images-2601.oke.minds.io/nemofin/blog/sample-blog-1526667225598005253

Deployment Considerations

Standard deployment.

Regression Scope

Blog images.

Platform Affected (web, mobile, etc)

Web.

Developer Testing Completed

Manual, Unit (PHPSpec)

Screenshots / Screen Recording

simplescreenrecorder-2023-07-17_14.24.10

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
Edited by Ben

Merge request reports