Add comments to clarify indexOffset logic

Expensify · Mar 17, 2021 · f085f11 · f085f11
1 parent fb34951
commit f085f11
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/src/pages/iou/steps/IOUParticipantsPage/IOUParticipantsRequest.js b/src/pages/iou/steps/IOUParticipantsPage/IOUParticipantsRequest.js
@@ -64,7 +64,7 @@ class IOUParticipantsRequest extends Component {
             title: 'CONTACTS',
             data: this.state.personalDetails,
             shouldShow: this.state.personalDetails.length > 0,
-            indexOffset: sections.reduce((prev, {data}) => prev + data.length, 0),
+            indexOffset: 0,
         });
 
         if (this.state.userToInvite) {

diff --git a/src/pages/iou/steps/IOUParticipantsPage/IOUParticipantsSplit.js b/src/pages/iou/steps/IOUParticipantsPage/IOUParticipantsSplit.js
@@ -109,13 +109,19 @@ class IOUParticipantsSplit extends Component {
             title: 'RECENTS',
             data: this.state.recentReports,
             shouldShow: this.state.recentReports.length > 0,
+
+            // takes the sum off the length of all data
+            // (this.state.selectedOptions) in previous sections
             indexOffset: sections.reduce((prev, {data}) => prev + data.length, 0),
         });
 
         sections.push({
             title: 'CONTACTS',
             data: this.state.personalDetails,
             shouldShow: this.state.personalDetails.length > 0,
+
+            // takes the sum off the length of all data
+            // (this.state.selectedOptions, this.state.recentReports) in previous sections
             indexOffset: sections.reduce((prev, {data}) => prev + data.length, 0),
         });
 

diff --git a/src/pages/iou/steps/IOUParticipantsPage/index.js b/src/pages/iou/steps/IOUParticipantsPage/index.js
@@ -48,7 +48,6 @@ const defaultProps = {
 };
 
 const IOUParticipantsPage = (props) => {
-    console.debug(props.participants);
     if (props.iou.loading) {
         return (
             <View style={styles.pageWrapper}>