PR comments.

packages/client/src/components/app/forms/BBReferenceField.svelte

@@ -1,17 +1,31 @@
 <script>
   import RelationshipField from "./RelationshipField.svelte"
   import { sdk } from "@budibase/shared-core"
+
+  export let defaultValue
+
+  function updateUserIDs(value) {
+    if (Array.isArray(value)) {
+      return value.map(val => sdk.users.getGlobalUserID(val))
+    } else {
+      return sdk.users.getGlobalUserID(value)
+    }
+  }
+
+  function updateReferences(value) {
+    if (sdk.users.containsUserID(value)) {
+      return updateUserIDs(value)
+    }
+    return value
+  }
 </script>
 
 <RelationshipField
   {...$$props}
   datasourceType={"user"}
   primaryDisplay={"email"}
-  valueConversion={value => {
+  defaultValue={updateReferences(defaultValue)}
-    if (Array.isArray(value)) {
+  onChange={value => {
-      return value.map(val => sdk.users.getGlobalUserID(val))
+    return updateReferences(value)
-    } else {
-      return sdk.users.getGlobalUserID(value)
-    }
   }}
 />

packages/client/src/components/app/forms/RelationshipField.svelte

@@ -18,7 +18,6 @@
   export let filter
   export let datasourceType = "table"
   export let primaryDisplay
-  export let valueConversion
 
   let fieldState
   let fieldApi
@@ -162,7 +161,7 @@
     const changed = fieldApi.setValue(value)
     if (onChange && changed) {
       onChange({
-        value: valueConversion ? valueConversion(value) : value,
+        value,
       })
     }
   }
@@ -179,9 +178,7 @@
   {field}
   {disabled}
   {validation}
-  defaultValue={valueConversion
+  defaultValue={expandedDefaultValue}
-    ? valueConversion(expandedDefaultValue)
-    : expandedDefaultValue}
   {type}
   bind:fieldState
   bind:fieldApi

packages/shared-core/src/sdk/documents/users.ts

@@ -84,3 +84,10 @@ export function getGlobalUserID(userId?: string): string | undefined {
   }
   return userId.split(prefix)[1]
 }
+
+export function containsUserID(value: string | undefined): boolean {
+  if (typeof value !== "string") {
+    return false
+  }
+  return value.includes(`${DocumentType.USER}${SEPARATOR}`)
+}