[HOLD for payment 2023-10-30] Remove and fix some TODOs that exist in the code

[tgolen]

Problem

Some TODOs exist in our code that are not helpful or are not consistently labeled.

This is coming from my large refactoring PR in an attempt to break it down into smaller pieces

Why this is important

It creates confusion and inconsistency

Solution

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01329374af863b4abc
• Upwork Job ID: 1714682014816800768
• Last Price Increase: 2023-10-18

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.88-11 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Remove and fix some TODOs that exist in the code #29880

If no regressions arise, payment will be issued on 2023-10-30. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

• Merged PR within 3 business days of assignment - 50% bonus
• Merged PR more than 9 business days after assignment - 50% penalty

[tgolen]

This was done internally so there are no payments.