Page MenuHomePhabricator

Remove CAN_JOIN policy from Conpherence
ClosedPublic

Authored by chad on Apr 13 2017, 4:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 18, 8:09 AM
Unknown Object (File)
Tue, Jun 17, 7:22 AM
Unknown Object (File)
Sat, Jun 14, 9:50 PM
Unknown Object (File)
Sat, Jun 14, 4:25 PM
Unknown Object (File)
May 18 2025, 9:03 AM
Unknown Object (File)
Apr 26 2025, 10:48 AM
Unknown Object (File)
Apr 17 2025, 8:17 AM
Unknown Object (File)
Apr 11 2025, 9:15 AM
Subscribers

Details

Summary

Fixes T12178, Fixes T11704 Not sure this feature gets any use and I can't find a similar option in other software, so removing it I think simiplifies a number of things. Removes CAN_JOIN and joinable is basically now CAN_VIEW and !$participating. Also removed some old transaction strings for other policies. Don't seem used.

Test Plan

Create a new room, edit room policies, see changes. Log into second account, search for rooms, everything now is visible.

Diff Detail

Repository
rP Phabricator
Branch
remove-can-join (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16439
Build 21877: Run Core Tests
Build 21876: arc lint + arc unit

Event Timeline

Just in case, let's actually destroy the data in a separate change on a later date? I'm 99% sure this is safe but there is 0% chance we can roll this back if we destroy the data right now.

resources/sql/autopatches/20170413.conpherence.01.canjoin.sql
1–2

Let's not actually drop/remove the column for at least a week, just in case we get into trouble and want to revert this? We can't reasonably undo it after we DROP COLUMN since we lose all the data.

src/applications/conpherence/storage/ConpherenceThread.php
19–36

(Keep these too for now.)

50

And this one.

This revision is now accepted and ready to land.Apr 13 2017, 12:49 PM
  • re-wind dropping the column
This revision was automatically updated to reflect the committed changes.