Skip to content

Commit

Permalink
Fix more linting
Browse files Browse the repository at this point in the history
  • Loading branch information
mdimjasevic committed Jan 15, 2024
1 parent d272978 commit e030a6e
Showing 1 changed file with 19 additions and 13 deletions.
32 changes: 19 additions & 13 deletions services/galley/src/Galley/API/Teams.hs
Original file line number Diff line number Diff line change
Expand Up @@ -987,22 +987,28 @@ deleteTeamMember' lusr zcon tid remove mBody = do
else sizeBeforeDelete - 1
E.deleteUser remove
toNotify <-
getFeatureStatus @LimitedEventFanoutConfig DontDoAuth tid >>= (\case
FeatureStatusEnabled -> E.getBillingTeamMembers tid
FeatureStatusDisabled -> do
let filterFromMembers list =
view userId <$> filter (`hasPermission` SetBilling) (list ^. teamMembers)
filterFromMembers <$> getTeamMembersForFanout tid) . wsStatus
getFeatureStatus @LimitedEventFanoutConfig DontDoAuth tid
>>= ( \case
FeatureStatusEnabled -> E.getBillingTeamMembers tid
FeatureStatusDisabled -> do
let filterFromMembers list =
view userId <$> filter (`hasPermission` SetBilling) (list ^. teamMembers)
filterFromMembers <$> getTeamMembersForFanout tid
)
. wsStatus
Journal.teamUpdate tid sizeAfterDelete $ filter (/= remove) toNotify
pure TeamMemberDeleteAccepted
else do
getFeatureStatus @LimitedEventFanoutConfig DontDoAuth tid >>= (\case
FeatureStatusEnabled -> do
admins <- E.getTeamAdmins tid
uncheckedDeleteTeamMember lusr (Just zcon) tid remove (Left admins)
FeatureStatusDisabled -> do
mems <- getTeamMembersForFanout tid
uncheckedDeleteTeamMember lusr (Just zcon) tid remove (Right mems)) . wsStatus
getFeatureStatus @LimitedEventFanoutConfig DontDoAuth tid
>>= ( \case
FeatureStatusEnabled -> do
admins <- E.getTeamAdmins tid
uncheckedDeleteTeamMember lusr (Just zcon) tid remove (Left admins)
FeatureStatusDisabled -> do
mems <- getTeamMembersForFanout tid
uncheckedDeleteTeamMember lusr (Just zcon) tid remove (Right mems)
)
. wsStatus
pure TeamMemberDeleteCompleted

-- This function is "unchecked" because it does not validate that the user has the `RemoveTeamMember` permission.
Expand Down

0 comments on commit e030a6e

Please sign in to comment.