App
App copied to clipboard
Expensify
[$250] Some of the report previews don't show the receipt preview
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] 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: @youssef-lr Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1724166388296949
Action Performed:
- Go to staging.new.expensify.com
- Submit few expenses by scanning receipt
Expected Result:
All the report preview should show the receipt preview
Actual Result:
Some of the report previews don't show the receipt preview, have to open the report then navigate back to see them.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
- [ ] Android: Native
- [ ] Android: mWeb Chrome
- [ ] iOS: Native
- [ ] iOS: mWeb Safari
- [x] MacOS: Chrome / Safari
- [ ] MacOS: Desktop
Screenshots/Videos
Add any screenshot/video evidence
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~01eef70c2f6c535337
- Upwork Job ID: 1829179854819733890
- Last Price Increase: 2024-08-29
Issue Owner
Current Issue Owner: @aimane-chnaif
Current assignee @youssef-lr is eligible for the AutoAssignerNewDotQuality assigner, not assigning anyone new.
![melvin-bot[bot] avatar](https://avatars.githubusercontent.com/in/179547?v=4 "Published by a pub.dev verified publisher"){kind=small}
Triggered auto assignment to @isabelastisser (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
Job added to Upwork: https://www.upwork.com/jobs/~01eef70c2f6c535337
Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif (External)
Hi, I'm Julian from Callstack - expert agency - and I would like to work on this issue.
I think I can manage to work on this next week @muttmuure, but if someone would like to volunteer: I posted my thoughts here https://expensify.slack.com/archives/C049HHMV9SM/p1725558647007169?thread_ts=1724166388.296949&cid=C049HHMV9SM
@youssef-lr @isabelastisser @JKobrynski @aimane-chnaif this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif Huh... This is 4 days overdue. Who can take care of this?
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif 6 days overdue. This is scarier than being forced to listen to Vogon poetry!
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif 10 days overdue. Is anyone even seeing these? Hello?
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif 12 days overdue now... This issue's end is nigh!
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif Still overdue 6 days?! Let's take care of this!
Starting a discussion here https://expensify.slack.com/archives/C03TQ48KC/p1727731553397549
I'm done with the implementation and I think it turned out great, the new behavior is:
- Only transactions that have a "warning" violation are returned via OpenApp
- For chats / policy expense chats, the remaining transactions are now fetched during GetOlderActions/GetNewerActions only when the action is a REPORTPREVIEW
- For money request reports, we already send back the transactions in OpenReport
Will be working on tests now to make the PRs available for review.
Auth ready for review. The App PR was merged by mistake, I branched off of to create another PR, when that PR got merged git merged the other PR as well.
@youssef-lr, @isabelastisser, @JKobrynski, @aimane-chnaif Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!