Skip to content

Commit 2331dd2

Browse files
committed
apply suggestion from ancestral review
1 parent 01cca4e commit 2331dd2

File tree

1 file changed

+3
-5
lines changed
  • apps/meteor/app/apps/server/bridges

1 file changed

+3
-5
lines changed

apps/meteor/app/apps/server/bridges/users.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -132,9 +132,7 @@ export class AppUserBridge extends UserBridge {
132132
return true;
133133
}
134134

135-
const { status, statusText } = fields;
136-
delete fields.statusText;
137-
delete fields.status;
135+
const { status, statusText, ...updateFields } = fields;
138136

139137
if (status) {
140138
await Presence.setStatus(user.id, status as UserStatus, statusText);
@@ -152,9 +150,9 @@ export class AppUserBridge extends UserBridge {
152150
);
153151
}
154152

155-
await Users.updateOne({ _id: user.id }, { $set: fields as any });
153+
await Users.updateOne({ _id: user.id }, { $set: updateFields as any });
156154

157-
void notifyOnUserChange({ clientAction: 'updated', id: user.id, diff: fields });
155+
void notifyOnUserChange({ clientAction: 'updated', id: user.id, diff: updateFields });
158156

159157
return true;
160158
}

0 commit comments

Comments
 (0)