Conversation
Bundle sizes [mpdx-react]Compared against d2c5322 No significant changes found |
|
Preview branch generated at https://MPDX-9326.d3dytjb8adxkk5.amplifyapp.com |
dr-bizz
left a comment
There was a problem hiding this comment.
Something else we found was incorrect is the calculation on the 2nd checkbox, where users can take their normal percentage of 403(b) out of their ASR and contribute it to their Roth 403(b).
The calculation should be the total request minus any contributions the user made to their 403 (b). And then their normal percentage of 403(b) of that, otherwise you are taking that percentage out of what you are adding to the 403B back into the 403B.
Can we also make sure the percentage calculation is correct?
|
@dr-bizz I updated The Roth 403(b) percentage deduction to now deduct the |
dr-bizz
left a comment
There was a problem hiding this comment.
This works great! Thank you for making these changes.
Can you remove Claude as a co-author on commits?
This just reinforces that the dev should know and understand each line and shouldn't blame Claude.
Upon doing so, you can merge
Description
On ASR, the 7% 403b Contribution is taking 7% out of the amount just fine, but it is also taking the 7% out of "403(b) Contribution - Roth" and prob Tranditional as well, which it shouldn't be.
See the screenshots, the 7% contribution to the 403 should be $1,400, but it's not, as it's adding 7% of the $10,000 in the 403b.
Issue outlined in thread here.
Jira ticket available here.
Testing
Checklist:
/pr-reviewcommand locally and fixed any relevant suggestions