Closed kbecciv closed 1 year ago
Triggered auto assignment to @twisterdotcom (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)Very weird. It seems to be just taking the oldest open description:
Like in this instance, it's a split with two others, but the description is the description of a split with just the two from earlier:
Job added to Upwork: https://www.upwork.com/jobs/~01edb91d2bc99f8597
Current assignee @twisterdotcom is eligible for the External assigner, not assigning anyone new.
Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia (External
)
Triggered auto assignment to @techievivek (External
), see https://stackoverflow.com/c/expensify/questions/7972 for more details.
in Split bill detail page, the description shown is not matching with original
For MoneyRequestConfirmationList
we are taking IOU
from Onyx, that is global IOU.
https://github.com/Expensify/App/blob/0d1995a209218f721eb8ba123091b17d3fb29964/src/components/MoneyRequestConfirmationList.js#L361-L372 example - When I try to create new Split Bill, the IOU will be using its description for the value. It's being set at below -
https://github.com/Expensify/App/blob/0d1995a209218f721eb8ba123091b17d3fb29964/src/pages/iou/MoneyRequestDescriptionPage.js#L53-L56
Like we are passing the iouAmount
to MoneyRquestConfirmationList
component, we can pass the new prop iouComment
from SplitBillDetailsPage
.
setMoneyRequestDescription()
can be a alternative, however this change should be made carefully, because that value(IOU) is being used at many places. When we open the modal we can use setMoneyRequestDescription()
to set the value of description and upon closing we can reset it to empty string.Split bill detail page doesn't show the correct comment
When users update the description, it will be stored in IOU (Onyx). https://github.com/Expensify/App/blob/2157057bdc23aaf1eb48846c9a87cd33ff6645ae/src/pages/iou/MoneyRequestDescriptionPage.js#L54
We just reset this value when open spillBill modal again
In
We're getting comment from IOU in Onyx, so that's wrong value
IOU.setMoneyRequestDescription('');
in onConfirm
and onSendMoney
.Or we can reset comment when the component is unmounted
Then pass iouComment={props.iou.comment} below this line
const iouComment = lodashGet(reportAction, 'originalMessage.comment')
Closing as a dupe of this previously reported issue. Feel free to add proposals to that issue instead.
Nice! I think that was created after, but I'm happy to have one off the plate. 😄
Created after, but it was raised way earlier in Slack. Better to keep the original issue rather than remember to pay out the contributor -- else I would have happily left this to you :)
@tienifr Please review this comment: https://github.com/Expensify/App/issues/19870#issuecomment-1572682875
which is regarding a regression that can be caused if we use props.iouComment
instead of props.iou.comment
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue found when executing PRhttps://github.com/Expensify/App/pull/19390
Action Performed:
Expected Result:
Description from step 2 should be displayed
Actual Result:
Description from step 3 is displayed
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.20.1
Reproducible in staging?: Yes
Reproducible in production?: No
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://platform.applause.com/services/links/v1/external/308be96fbcf3f79db684db2be6e296224360728aeae1cc63681b405b0cea2035
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit