WHY THEM  Writes about code protectiveness and openness to refactoring in their own reviews.

“The goal here is to react to both `isPageFocused` and `hasNew` so making it an effect is the easiest way to do that. We might want to refactor polling more generally and move some of this state around but I'm hesitant to do this now.”

WHY THEM  Has reviewed this kind of change before.

“the name here is confusing. I saw that you define a new type here `apps/dove/modules/ordering/components/order-payments/order-payments.tsx` but it's confusing as it's not really an order. I think that would be clearer if order-payments component have a maxAmount props and internally calculate the remaining to pay wdyt ?”

WHY THEM  Has reviewed this kind of change before.

“why remove and not delete like the contractFile? why not making it required in the method argument?”