Skip to content

Commit 53748c7

Browse files
committed
fix some structural things and add to docs ignore
1 parent af243b7 commit 53748c7

File tree

4 files changed

+5
-5
lines changed

4 files changed

+5
-5
lines changed

src/sentry/api/serializers/models/organization.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -795,6 +795,8 @@ def serialize( # type: ignore[override]
795795
"streamlineOnly",
796796
"ingestThroughTrustedRelaysOnly",
797797
"enabledConsolePlatforms",
798+
"hasGranularReplayPermissions",
799+
"replayAccessMembers",
798800
]
799801
)
800802
class DetailedOrganizationSerializerWithProjectsAndTeamsResponse(

src/sentry/core/endpoints/organization_details.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -855,7 +855,7 @@ class OrganizationDetailsPutSerializer(serializers.Serializer):
855855
)
856856
replayAccessMembers = serializers.ListField(
857857
child=serializers.IntegerField(),
858-
help_text="A list of organization member IDs who have permission to access replay data. When empty, all members have access. Requires the granular-replay-permissions feature flag.",
858+
help_text="A list of organization member IDs who have permission to access replay data. Requires the hasGranularReplayPermissions flag to be true to be enforced.",
859859
required=False,
860860
allow_null=True,
861861
)

src/sentry/replays/endpoints/organization_replay_endpoint renamed to src/sentry/replays/endpoints/organization_replay_endpoint.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,7 @@ def check_replay_access(self, request: Request, organization: Organization) -> R
2121
Check if the session replay feature is enabled and user has replay permissions.
2222
Returns a Response object if access should be denied, None if access is granted.
2323
"""
24-
if not features.has(
25-
"organizations:session-replay", organization, actor=request.user
26-
):
24+
if not features.has("organizations:session-replay", organization, actor=request.user):
2725
return Response(status=404)
2826

2927
if not has_replay_permission(organization, request.user):

src/sentry/replays/permissions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@
55
from django.contrib.auth.models import AnonymousUser
66

77
from sentry import features
8+
from sentry.models.options.organization_option import OrganizationOption
89
from sentry.models.organizationmember import OrganizationMember
910
from sentry.models.organizationmemberreplayaccess import OrganizationMemberReplayAccess
10-
from sentry.models.organizationoption import OrganizationOption
1111

1212
if TYPE_CHECKING:
1313
from sentry.models.organization import Organization

0 commit comments

Comments
 (0)