Skip to content

Commit

Permalink
Improve reportActions explanation
Browse files Browse the repository at this point in the history
  • Loading branch information
puneetlath committed Jun 26, 2023
1 parent 312ec88 commit 0eb83f4
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/libs/migrations/PersonalDetailsByAccountID.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,8 +65,9 @@ export default function () {
}

// We migrate reportActions to have the new accountID-based data if they don't already.
// If we are not able to get the accountID for some reason, we will just clear the reportAction
// If we are not able to get the accountID from personalDetails for some reason, we will just clear the reportAction
// and let it be fetched from the API next time they open the report and scroll to that action.
// We do this because we know the reportAction from the API will include the needed accountID data.
_.each(oldReportActions, (reportActionsForReport, onyxKey) => {
if (_.isEmpty(reportActionsForReport)) {
Log.info(`[Migrate Onyx] Skipped migration PersonalDetailsByAccountID for ${onyxKey} because there were no reportActions`);
Expand Down

0 comments on commit 0eb83f4

Please sign in to comment.