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)
Fri, Dec 20, 6:57 PM
Unknown Object (File)
Thu, Dec 19, 10:36 PM
Unknown Object (File)
Sat, Nov 30, 4:22 AM
Unknown Object (File)
Wed, Nov 27, 2:09 AM
Unknown Object (File)
Nov 23 2024, 2:41 PM
Unknown Object (File)
Nov 18 2024, 8:28 PM
Unknown Object (File)
Nov 15 2024, 9:34 AM
Unknown Object (File)
Nov 14 2024, 10:11 PM
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).