Closed kbecciv closed 1 year ago
Triggered auto assignment to @mateocole (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)Dev: Web - Console error on split bill in #announce
https://github.com/Expensify/App/blob/7fea760c2bb7556815aaca2d61861bd1cf39d6b1/src/components/ReportActionItem/MoneyRequestAction.js#L48
hasOutstandingIOU
is a required field.
Remove isRequired
chatReport: PropTypes.shape({
/** Whether the chat report has an outstanding IOU */
hasOutstandingIOU: PropTypes.bool,
}),
N/A
Console error for split bill in #announce or any group chat
For all group reports(not only for #announce) the backend response is not setting the field hasOutstandingIOU
in the onyx data. With this in the MoneyRequestAction
component, we have the prop hasOutstandingIOU
required(PropTypes.bool.isRequired
) but for group reports its value is undefined which is causing the issue chatReport.hasOutstandingIOU` is marked as required in `MoneyRequestAction`, but its value is `undefined
.
We need to update the prop type by taking out the required(hasOutstandingIOU: PropTypes.bool
) in MoneyRequestAction
component.
https://github.com/Expensify/App/blob/a97ca9b2092c56e7d237e13e56a649cf08993090/src/components/ReportActionItem/MoneyRequestAction.js#L48
@mateocole Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
Triggered auto assignment to @roryabraham (Engineering
), see https://stackoverflow.com/c/expensify/questions/4319 for more details.
Hey @roryabraham could you see if you get this same console error in Dev?
Saw this console warning on the workspace invite page:
Could not reproduce the error shown in the videos above however.
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
It should not show any console error
Actual Result:
console error appears(Failed prop type: The prop
chatReport.hasOutstandingIOU
is marked as required inMoneyRequestAction
, but its value isundefined
)Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: Dev 1.3.43-2 Reproducible in staging?: n/a Reproducible in production?: n/a If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/0b7ac7b3-219c-490b-ac27-cf22aa10c6c6
Expensify/Expensify Issue URL: Issue reported by: @gadhiyamanan Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689863593169429
View all open jobs on GitHub