added Sprint::09/11 - Nuanced Numbat Squad::Green scoped labels
added 1 commit
- 01a69464 - All the permissions* (minus all the things I forgot/don't now about)
unmarked as a Work In Progress
changed target branch from
master
toepic/permissions-28
added 1 commit
- 2659fb93 - Fixing rate limit unit test that was time dependent
added 1 commit
- fe2dc343 - Refactoring flags out into their own class so we can iterate over the...
added 1 commit
- 9e615349 - Helpers/flags is now trying to check the type property on FeedSyncEntity. Need...
approved this merge request
approved this merge request
unapproved this merge request
added 9 commits
- 9e615349...26e56438 - 7 commits from branch
epic/permissions-28
- 8ab258df - Merge remote-tracking branch 'origin/master' into feat/permissions-on-entities-737
- 40bd92e8 - Unit tests for Manager
- 9e615349...26e56438 - 7 commits from branch
changed title from Feat/permissions on entities 737 to (WIP) Feat/permissions on entities 737
approved this merge request
added 23 commits
- 95e7b86a - (chore): increase presigned video request to 90 days
- b0c1432e - (fix): allow video to played if only 50% of transcodes available
- 400bb77c - Return error if access by unknown user.
- b69e1e50 - WIP Testing permissions on exports
- f390f542 - Revert "(chore): increase presigned video request to 90 days"
- 0028f4c6 - (fix): usa and japan stripe fixes
- eaaf74d0 - (fix): us customers are required to request capabilities
- 997ec821 - (feat): support recording multiple entity types
- 719eb4a4 - Sprint/NuancedNumbat: Null paging token #866 (closed)
- 1071aa8f - Merge branch 'fix/null-paging-token-866' into 'master'
- 2ec16e85 - [Sprint/NuancedNumbat] (fix): Logout should provide $session value - Error:...
- d8446ca6 - Merge branch 'fix/sessions-issue-773' into 'master'
- 34594d7d - (fix): still respect activity metrics in addition to others
- 9fec7059 - Sprint/NuancedNumbat: Added in function to return original owner guid #806 (closed)
- 2203eec7 - Merge branch 'fix/group-report-806' into 'master'
- a3dfc58d - (chore) Add missing spec tests for new classes; Refactor usage to not use...
- 65b881df - Merge branch 'chore/616-propogate-props' into 'master'
- 125e0da2 - (fix): boosts
- 12fda64d - (chore): september email
- 8a401b6a - (fix): 2000 not 5000 for email
- 4f6b8f18 - Merge branch 'fix/null-user-email-setting-872' into 'master'
- c3608afc - (fix): do not request balance if not user provided
- 9c3c4a85 - Merge remote-tracking branch 'origin/master' into feat/permissions-on-entities-737
Toggle commit listmarked as a Work In Progress from b69e1e50
- Resolved by Brian Hatchet
- Resolved by Brian Hatchet
- Resolved by Brian Hatchet
- Resolved by Brian Hatchet
92 92 $response['entity']['canEdit'] = $entity->canEdit(); 93 93 Security\ACL::$ignore = $ignore; 94 94 } 95 96 95 97 } 96 98 } 97 99 100 $currentUser = Core\Session::getLoggedinUser(); - Developer
do we need this now that we're including permissions on the export itself?
112 112 113 113 // Trim to correct length 114 114 $entities = array_slice($entities, 0, $limit); 115 $permissions = null; 116 $currentUser = Core\Session::getLoggedinUser(); - Developer
do we need this now that we're including permissions on the export itself?
33 { 34 $activity = new Activity($pages[1]); 35 36 if (!Security\ACL::_()->read($activity)) { 37 return Factory::response([ 38 'status' => 'error', 39 'message' => 'You do not have permission to view this post' 40 ]); 41 } 42 43 if (!$activity->guid || Helpers\Flags::shouldFail($activity)) { 44 return Factory::response(['status' => 'error']); 45 } 46 47 $response = [ 48 'activtiy' => $activity->export() - Developer
typo here!
37 return Factory::response([ 38 'status' => 'error', 39 'message' => 'You do not have permission to view this post' 40 ]); 41 } 42 43 if (!$activity->guid || Helpers\Flags::shouldFail($activity)) { 44 return Factory::response(['status' => 'error']); 45 } 46 47 $response = [ 48 'activtiy' => $activity->export() 49 ]; 50 51 //Calculate new permissions object with the entities 52 if ($activity && Di::_()->get('Features\Manager')->has('permissions')) { - Developer
do we need this now that we're including permissions on the export itself?
151 151 ]); 152 152 } 153 153 154 $permissions = null; 155 //Calculate new permissions object with the entities 156 if ($boosts && Di::_()->get('Features\Manager')->has('permissions')) { - Developer
do we need this now that we're including permissions on the export itself?
31 34 ->setUser(Session::getLoggedinUser() ?: null) 32 35 ->setUrns($urns) 33 36 ->setOpts([ 34 'asActivities' => $asActivities 37 'asActivities' => $asActivities, 35 38 ]); 36 39 37 40 $entities = $resolver->fetch(); 38 41 42 $permissions = null; 43 //Calculate new permissions object with the entities 44 if (Di::_()->get('Features\Manager')->has('permissions')) { - Developer
do we need this now that we're including permissions on the export itself?
198 196 } 199 197 } 200 198 199 $permissions = null; 200 //Calculate new permissions object with the entities 201 if (Di::_()->get('Features\Manager')->has('permissions')) { - Developer
do we need this now that we're including permissions on the export itself?
157 154 } 158 155 } 159 156 157 $permissions = null; 158 //Calculate new permissions object with the entities 159 if (Di::_()->get('Features\Manager')->has('permissions')) { - Developer
do we need this now that we're including permissions on the export itself?
122 120 } 123 121 } 124 122 123 $permissions = null; 124 //Calculate new permissions object with the entities 125 if (Di::_()->get('Features\Manager')->has('permissions')) { - Developer
do we need this now that we're including permissions on the export itself?
added 6 commits
- 9c3c4a85...1adc33a5 - 5 commits from branch
epic/permissions-28
- 3cff4b90 - Merge branch 'epic/permissions-28' into feat/permissions-on-entities-737
- 9c3c4a85...1adc33a5 - 5 commits from branch