Page MenuHomePhabricator

Add Spaces support to Phriction
ClosedPublic

Authored by epriestley on Jul 30 2018, 11:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 7, 12:02 AM
Unknown Object (File)
Sat, Apr 6, 2:06 AM
Unknown Object (File)
Sun, Mar 31, 6:51 PM
Unknown Object (File)
Mar 7 2024, 12:09 AM
Unknown Object (File)
Feb 18 2024, 2:06 AM
Unknown Object (File)
Feb 3 2024, 7:11 PM
Unknown Object (File)
Dec 26 2023, 12:53 PM
Unknown Object (File)
Dec 24 2023, 3:08 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13164. See PHI774. Fixes T12435.

Since Phriction is hierarchical, there isn't a super strong motivation to support Spaces: you can generally set policies on a small number of documents to get the desired effective policy behavior.

However, it still improves consistency and there's no reason not to support Spaces. In the case where you have some moderately weird/complex policy on one or more Spaces, using Spaces to define the policy behavior can make things a bit simpler and easier to understand.

This probably doesn't actually fix whatever the root problem in T12435 was (complicated, non-hierarchical access policies?). See also a bunch of discussion in T12442. So we might end up going beyond this to address other use cases, but I think this is reasonable regardless.

Test Plan

Created and edited Phriction documents and shifted them between Spaces. Searched by Space, etc.

Diff Detail

Repository
rP Phabricator
Branch
wikispace1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20543
Build 27904: Run Core Tests
Build 27903: arc lint + arc unit

Event Timeline

Owners added a subscriber: Restricted Owners Package.Jul 30 2018, 11:32 PM
This revision is now accepted and ready to land.Jul 31 2018, 4:45 PM
This revision was automatically updated to reflect the committed changes.