Adds back commented out code and fixing any code that is still flagged as problematic.
This was a little trickier to clean up existing code to be conformant, the biggest one is probably type guards, which I will concede is probably not the best handled, but that seemed to be what was recommended to clean out the anys in that situation.
Related Issue
Fixes #2783
Acceptance Criteria
Please copy the acceptance criteria from your ticket and paste it here for your reviewer(s)
For frontend PR’s - include a description (including anything that’s out of scope) for what you want the designers to review
ex: “Check out the whitespace on the page, as well as the dropdowns in the form. Here is the corresponding Figma link. You can ignore everything else. Also, the button at the bottom doesn’t work now”
Additional Information
Anything else the review team should know?
Checklist
[ ] If this code affects the other scrum team, have they been notified? (In Slack, as reviewers, etc.)
PULL REQUEST
Summary
Adds back commented out code and fixing any code that is still flagged as problematic.
This was a little trickier to clean up existing code to be conformant, the biggest one is probably type guards, which I will concede is probably not the best handled, but that seemed to be what was recommended to clean out the
any
s in that situation.Related Issue
Fixes #2783
Acceptance Criteria
Please copy the acceptance criteria from your ticket and paste it here for your reviewer(s)
For frontend PR’s - include a description (including anything that’s out of scope) for what you want the designers to review
Additional Information
Anything else the review team should know?
Checklist