Skip to content

Commit 17fe480

Browse files
committed
remove str calls
1 parent cb6b9e4 commit 17fe480

File tree

1 file changed

+2
-4
lines changed
  • src/sentry/api/serializers/models/organization_member

1 file changed

+2
-4
lines changed

src/sentry/api/serializers/models/organization_member/base.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,7 @@ def get_attrs(
4545
email_map: MutableMapping[str, str] = {}
4646
for u in user_service.serialize_many(filter={"user_ids": users_set}):
4747
# Filter out the emails from the user data
48-
if "emails" in u:
49-
del u["emails"]
48+
u.pop("emails", None)
5049
users_by_id[u["id"]] = u
5150
email_map[u["id"]] = u["email"]
5251

@@ -109,7 +108,6 @@ def serialize(
109108

110109
# helping mypy - we know email will never be None based on the model
111110
assert email is not None
112-
email = str(email)
113111

114112
inviter_name = None
115113
if obj.inviter_id:
@@ -118,7 +116,7 @@ def serialize(
118116
inviter_name = inviter.get_display_name()
119117

120118
# helping mypy - we know name will be a string since we fall back to email
121-
name = str(serialized_user["name"] if serialized_user else email)
119+
name = serialized_user["name"] if serialized_user else email
122120

123121
data: OrganizationMemberResponse = {
124122
"id": str(obj.id),

0 commit comments

Comments
 (0)