Open m-natarajan opened 6 days ago
Triggered auto assignment to @stephanieelliott (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989
@m-natarajan Please try cleaning your Onyx cache, rerun then app and tell us if this fixes the problem. This solution worked for my case.
If yes then I believe this should not be a big problem, unfortunately we're still in development mode for the Search features so small bumps like this one might happen.
CC @luacmartins for visibility
App Crash when opening search page
We are accessing shouldShowCategoryColumn
without using optional chaining operator. This causes a crash sometimes if the network is slow and SearchResults
object has not fully loaded.
We should use optionalChaining when accessing this property:
metadata?.columnsToShow?.shouldShowCategoryColumn,
We will use optionalChaining when acessing shouldShowCategoryColumn
in these places:
https://github.com/Expensify/App/blob/2f688ad31969f148b18a3793b057b31c4f02aec7/src/libs/SearchUtils.ts#L141
We will optionalChaining when accessing shouldShowTagColumn
and shouldShowTaxColumn
here:
If yes then I believe this should not be a big problem, unfortunately we're still in development mode for the Search features so small bumps like this one might happen.
Agree, but I think we should add optional chaining as a sanity check for scenarios like these, so the app doesn't crash!
Job added to Upwork: https://www.upwork.com/jobs/~0102a0367add306853
Triggered auto assignment to Contributor-plus team member for initial proposal review - @abdulrahuman5196 (External
)
Upwork job price has been updated to $125
📣 @abdulrahuman5196 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!
@neonbhai please put up a PR to add optional chaining
📣 @neonbhai 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻 Keep in mind: Code of Conduct | Contributing 📖
@neonbhai how's the PR coming along?
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: Reproducible in staging?: Needs reproduction Reproducible in production?: Needs reproduction 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 Expensify/Expensify Issue URL: Issue reported by: @roryabraham Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1719267975652129
Action Performed:
Expected Result:
Search page opens
Actual Result:
App crash on opening search page
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @abdulrahuman5196