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)
Sun, Mar 3, 7:30 AM
Unknown Object (File)
Feb 21 2024, 1:39 AM
Unknown Object (File)
Feb 11 2024, 2:17 AM
Unknown Object (File)
Jan 6 2024, 9:32 AM
Unknown Object (File)
Dec 27 2023, 10:37 AM
Unknown Object (File)
Dec 27 2023, 10:37 AM
Unknown Object (File)
Dec 27 2023, 10:37 AM
Unknown Object (File)
Dec 27 2023, 10:37 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 16442
Build 21882: Run Core Tests
Build 21881: 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 ↗(On Diff #42508)

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.