Skip to content

Commit

Permalink
chore(data-secrecy): Remove bitflag update from Org Details Endpoint (#…
Browse files Browse the repository at this point in the history
…75449)

Need to gate this behind a plan, so want to make sure noone can update
their bitflag before then. checked redash and isn't the case but ill add
this again once i have the permanant feature flag setup
  • Loading branch information
iamrajjoshi committed Aug 1, 2024
1 parent 14c258e commit 95a3ace
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
3 changes: 0 additions & 3 deletions src/sentry/api/endpoints/organization_details.py
Original file line number Diff line number Diff line change
Expand Up @@ -512,8 +512,6 @@ def save(self):
org.flags.require_email_verification = data["requireEmailVerification"]
if "allowMemberProjectCreation" in data:
org.flags.disable_member_project_creation = not data["allowMemberProjectCreation"]
if "allowSuperuserAccess" in data:
org.flags.prevent_superuser_access = not data["allowSuperuserAccess"]
if "name" in data:
org.name = data["name"]
if "slug" in data:
Expand All @@ -531,7 +529,6 @@ def save(self):
"require_2fa": org.flags.require_2fa.is_set,
"codecov_access": org.flags.codecov_access.is_set,
"disable_member_project_creation": org.flags.disable_member_project_creation.is_set,
"prevent_superuser_access": org.flags.prevent_superuser_access.is_set,
},
}

Expand Down
4 changes: 0 additions & 4 deletions tests/sentry/api/endpoints/test_organization_details.py
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,6 @@ def test_various_options(self, mock_get_repositories):

assert org.flags.early_adopter
assert org.flags.codecov_access
assert org.flags.prevent_superuser_access
assert not org.flags.allow_joinleave
assert org.flags.disable_shared_issues
assert org.flags.enhanced_privacy
Expand Down Expand Up @@ -503,9 +502,6 @@ def test_various_options(self, mock_get_repositories):
assert "to {}".format(data["openMembership"]) in log.data["allow_joinleave"]
assert "to {}".format(data["isEarlyAdopter"]) in log.data["early_adopter"]
assert "to {}".format(data["codecovAccess"]) in log.data["codecov_access"]
assert (
"to {}".format(not data["allowSuperuserAccess"]) in log.data["prevent_superuser_access"]
)
assert "to {}".format(data["enhancedPrivacy"]) in log.data["enhanced_privacy"]
assert "to {}".format(not data["allowSharedIssues"]) in log.data["disable_shared_issues"]
assert "to {}".format(data["require2FA"]) in log.data["require_2fa"]
Expand Down

0 comments on commit 95a3ace

Please sign in to comment.