Merge pull request Expensify#29880 from Expensify/tgolen-todos

Remove and fix some TODOs that exist in the code

src/libs/Navigation/OnyxTabNavigator.js

@@ -6,13 +6,13 @@ import Tab from '../actions/Tab';
 import ONYXKEYS from '../../ONYXKEYS';
 
 const propTypes = {
-    /* ID of the tab component to be saved in onyx */
+    /** ID of the tab component to be saved in onyx */
     id: PropTypes.string.isRequired,
 
-    /* Name of the selected tab */
+    /** Name of the selected tab */
     selectedTab: PropTypes.string,
 
-    /* Children nodes */
+    /** Children nodes */
     children: PropTypes.node.isRequired,
 };
 

src/libs/actions/BankAccounts.ts

@@ -145,7 +145,7 @@ function connectBankAccountWithPlaid(bankAccountID: number, selectedPlaidBankAcc
 /**
  * Adds a bank account via Plaid
  *
- * @TODO offline pattern for this command will have to be added later once the pattern B design doc is complete
+ * TODO: offline pattern for this command will have to be added later once the pattern B design doc is complete
  */
 function addPersonalBankAccount(account: PlaidBankAccount) {
     const commandName = 'AddPersonalBankAccount';

src/pages/DetailsPage.js

@@ -89,8 +89,6 @@ function DetailsPage(props) {
     let details = _.find(props.personalDetails, (detail) => detail.login === login.toLowerCase());
 
     if (!details) {
-        // TODO: these personal details aren't in my local test account but are in
-        // my staging account, i wonder why!
         if (login === CONST.EMAIL.CONCIERGE) {
             details = {
                 accountID: CONST.ACCOUNT_ID.CONCIERGE,