Repository navigation
IOU Request - Error "Invalid onyxMethod mergecollection in Onyx update." appeared in Console #10001
Description
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:
- Sign into NewDot as user 1
- Start a chat with a user 2 (a user you have not chatted with before)
- Create an IOU request (either from within the chat, or from the LHN menu)
- Tap the IOUPreview to open the IOU Details Modal
Expected Result:
No error in Console logs.
Actual Result:
Error "Invalid onyxMethod mergecollection in Onyx update." appeared in Console
Workaround:
Unknown
Platform:
Where is this issue occurring?
- Web
Version Number: v1.1.85-2
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Activity
OSBotify commentedon Jul 19, 2022
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
StagingDeployCashdeploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:melvin-bot commentedon Jul 19, 2022
Triggered auto assignment to @neil-marcellini (
Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.mountiny commentedon Jul 19, 2022
cc @Julesssss
neil-marcellini commentedon Jul 19, 2022
It looks like we're not using the latest version of Onyx since the check was recently updated here.
mountiny commentedon Jul 19, 2022
Ah, we have bumped to 1.0.8 here #9877, but that used
mergeCollectionand then the update tomergecollectionhas been in later version, which failed to bundle Expensify/react-native-onyx#160.So we should update to 1.0.10 :/ Expensify/react-native-onyx#162
@neil-marcellini are you able to create the PR and run npm install?
My mac always changes the integrity to sha-1 because intel 😡
neil-marcellini commentedon Jul 19, 2022
Yep I was just about to say that. I'll have a PR up very soon. I might have some integrity changes as well but I'll have to see. Could we manually omit those changes?
mountiny commentedon Jul 19, 2022
I am not sure tbh :/ if it is just a few of them then it is fine. But for me, AFAIK because of the intel having different architecture it creates like hundreds lines big diff :/
Julesssss commentedon Jul 20, 2022
Thanks for fixing this @neil-marcellini!