From 114a54f5246b9ccab2ae1abd79b32b2622be2c7b Mon Sep 17 00:00:00 2001
From: Antoine Rosenbach <arosenbach@salesforce.com>
Date: Wed, 25 Oct 2023 11:09:29 +0200
Subject: [PATCH] add some comments

---
 .../recordPickerDynamicTarget.html                         | 1 +
 .../recordPickerDynamicTarget/recordPickerDynamicTarget.js | 7 +++++++
 .../default/lwc/recordPickerHello/recordPickerHello.js     | 3 +++
 3 files changed, 11 insertions(+)

diff --git a/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.html b/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.html
index bb1e95430..73cd06ace 100644
--- a/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.html
+++ b/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.html
@@ -8,6 +8,7 @@
             <label class="slds-form-element__label">Select a record</label>
             <div class="slds-form-element__control slds-combobox-group">
                 <template if:true={showTargetSelector}>
+                    <!-- Target selector -->
                     <lightning-combobox
                         data-id="targetSelector"
                         label="Select a target sObject"
diff --git a/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.js b/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.js
index d1f76fa12..c6f5df8f9 100644
--- a/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.js
+++ b/force-app/main/default/lwc/recordPickerDynamicTarget/recordPickerDynamicTarget.js
@@ -1,5 +1,9 @@
 import { LightningElement } from 'lwc';
 
+// As of today, `lightning-record-picker` supports a single targets.
+// This sample component shows how you can turn `lightning-record-picker` into
+// a multi-target record picker, where the user can select the target object
+// before searching.
 export default class RecordPickerDynamicTarget extends LightningElement {
     targetObjects = [
         {
@@ -45,6 +49,9 @@ export default class RecordPickerDynamicTarget extends LightningElement {
     }
 
     handleTargetSelection(event) {
+        // Prevent lightning-combobox `change` event from bubbling
+        event.stopPropagation();
+
         this.selectedTarget = event.target.value;
         this.refs.recordPicker.clearSelection();
     }
diff --git a/force-app/main/default/lwc/recordPickerHello/recordPickerHello.js b/force-app/main/default/lwc/recordPickerHello/recordPickerHello.js
index b7684b9e3..57899e2f5 100644
--- a/force-app/main/default/lwc/recordPickerHello/recordPickerHello.js
+++ b/force-app/main/default/lwc/recordPickerHello/recordPickerHello.js
@@ -16,6 +16,9 @@ export default class RecordPickerHello extends LightningElement {
         this.selectedRecordId = event.detail.recordId;
     }
 
+    // A GraphQL query is sent after the record picker change event has been dispatched.
+    // This is the recommended practice for fetching record fields based on
+    // the record picker change event's recordId.
     @wire(graphql, {
         query: gql`
             query searchContacts($selectedRecordId: ID) {