Remove references to <item>._id.$oid in app code now they are handled by serializer (#1362)

Author: ml-evs

CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## v0.6.4 (September 2025)
+
+This patch release simply fixes a few UI bugs introduced in v0.6.3 (and earlier).
+It also signifies the adoption of the Contributor Covenant Code of Conduct (v2).
+
+### What's Changed
+
+* Broken admin dashboard UI for user management by @ml-evs in #1361
+* Inability to insert new items created via copying into a collection by @BenjaminCharmes in #1356
+* Long message dialog box formatting by @BenjaminCharmes in #1346
+
+**Full Changelog**: https://github.com/datalab-org/datalab/compare/v0.6.3...v0.6.4
+
 ## v0.6.3 (September 2025)
 
 This patch release primarily improves block serialization performance and extensibility, as well as improving error handling for both developers and users.

webapp/src/components/UserTable.vue

@@ -33,7 +33,7 @@
           <select
             v-model="user.role"
             class="dropdown"
-            @change="confirmUpdateUserRole(user._id.$oid, $event.target.value)"
+            @change="confirmUpdateUserRole(user._id, $event.target.value)"
           >
             <option value="user">User</option>
             <option value="admin">Admin</option>
@@ -44,21 +44,21 @@
           <button
             v-if="user.account_status === 'active'"
             class="btn btn-outline-danger btn-sm text-uppercase text-monospace"
-            @click="confirmUpdateUserStatus(user._id.$oid, 'deactivated')"
+            @click="confirmUpdateUserStatus(user._id, 'deactivated')"
           >
             Deactivate
           </button>
           <button
             v-else-if="user.account_status === 'unverified'"
             class="btn btn-outline-success btn-sm text-uppercase text-monospace"
-            @click="confirmUpdateUserStatus(user._id.$oid, 'active')"
+            @click="confirmUpdateUserStatus(user._id, 'active')"
           >
             Activate
           </button>
           <button
             v-else-if="user.account_status === 'deactivated'"
             class="btn btn-outline-success btn-sm text-uppercase text-monospace"
-            @click="confirmUpdateUserStatus(user._id.$oid, 'active')"
+            @click="confirmUpdateUserStatus(user._id, 'active')"
           >
             Activate
           </button>
@@ -93,14 +93,14 @@ export default {
       }
     },
     async confirmUpdateUserRole(user_id, new_role) {
-      const originalCurrentUser = this.original_users.find((user) => user._id.$oid === user_id);
+      const originalCurrentUser = this.original_users.find((user) => user._id === user_id);
 
       if (originalCurrentUser.role === "admin") {
         DialogService.error({
           title: "Role Change Error",
           message: "You can't change an admin's role.",
         });
-        this.users.find((user) => user._id.$oid === user_id).role = originalCurrentUser.role;
+        this.users.find((user) => user._id === user_id).role = originalCurrentUser.role;
         return;
       }
 
@@ -112,22 +112,22 @@ export default {
       if (confirmed) {
         await this.updateUserRole(user_id, new_role);
       } else {
-        this.users.find((user) => user._id.$oid === user_id).role = originalCurrentUser.role;
+        this.users.find((user) => user._id === user_id).role = originalCurrentUser.role;
       }
     },
     async confirmUpdateUserStatus(user_id, new_status) {
-      const originalCurrentUser = this.original_users.find((user) => user._id.$oid === user_id);
+      const originalCurrentUser = this.original_users.find((user) => user._id === user_id);
 
       const confirmed = await DialogService.confirm({
         title: "Change User Status",
         message: `Are you sure you want to change ${originalCurrentUser.display_name}'s status from "${originalCurrentUser.account_status}" to "${new_status}"?`,
         type: "warning",
       });
       if (confirmed) {
-        this.users.find((user) => user._id.$oid == user_id).account_status = new_status;
+        this.users.find((user) => user._id == user_id).account_status = new_status;
         await this.updateUserStatus(user_id, new_status);
       } else {
-        this.users.find((user) => user._id.$oid === user_id).account_status =
+        this.users.find((user) => user._id === user_id).account_status =
           originalCurrentUser.account_status;
       }
     },

webapp/src/file_upload.js

@@ -68,7 +68,7 @@ export default function setupUppy(item_id, trigger_selector, reactive_file_list)
           file_id: response_body.file_id,
           file_info: {
             ...response_body.file_information,
-            immutable_id: response_body.file_information.immutable_id.$oid,
+            immutable_id: response_body.file_information.immutable_id,
           },
         });
       }