Page MenuHomePhorge

Remove Chatlog entirely
ClosedPublic

Authored by Matthew on Nov 27 2023, 17:50.
Tags
None
Referenced Files
F2193528: D25480.1716000346.diff
Fri, May 17, 02:45
Unknown Object (File)
Tue, May 14, 12:33
Unknown Object (File)
Mon, May 13, 18:26
Unknown Object (File)
Mon, May 13, 18:26
Unknown Object (File)
Mon, May 13, 18:26
Unknown Object (File)
Mon, May 13, 18:26
Unknown Object (File)
Mon, May 13, 18:26
Unknown Object (File)
Mon, May 13, 18:26

Details

Summary

This commit removes ChatLog entirely. All of the application files are removed, and the migrations used are stubbed out. I stubbed the migrations as that allows for existing installs to make no changes, but new installs will not create the database.

Fixes T15126

Test Plan

Loaded up http://phorge.local/chatlog and confirmed the 404. Loaded up http://phorge.local/applications/view/PhabricatorChatLogApplication and confirmed the 404. Created a new database prefix and ran bin/storage upgrade against it, confirmed that the chatlog database was not created. Restored another prefix (an old one) and ran bin/storage upgrade and confirmed database was not deleted.

Diff Detail

Repository
rP Phorge
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 956
Build 956: arc lint + arc unit