Page MenuHomePhabricator

Fix old image macros and memes for logged out users
ClosedPublic

Authored by epriestley on Sep 4 2014, 4:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 7:00 AM
Unknown Object (File)
Mon, Apr 8, 6:07 PM
Unknown Object (File)
Sun, Apr 7, 7:28 AM
Unknown Object (File)
Sun, Mar 24, 2:54 AM
Unknown Object (File)
Mar 4 2024, 6:40 PM
Unknown Object (File)
Mar 4 2024, 6:30 AM
Unknown Object (File)
Mar 4 2024, 6:30 AM
Unknown Object (File)
Feb 13 2024, 7:16 AM
Subscribers
Tokens
"Haypence" token, awarded by chad.

Details

Summary

Fixes T6013. Old image macros/memes never had the file edge written.

We also never wrote file edges for audio.

Finally, the meme controller didn't allow public access.

Write edges for images and audio, perform a migration to populate the historic ones, and make the Editor keep them up to date going forward.

Test Plan
  • Updated image, saw new image attach and old image detach.
  • Updated audio, saw new audio attach and old audio detach.
  • Ran migration.
  • Viewed memes as a logged-out user.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Fix old image macros and memes for logged out users.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Sep 4 2014, 7:29 PM
epriestley updated this revision to Diff 25068.

Closed by commit rP8038af4bd570 (authored by @epriestley).