Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
2.99k stars 2.5k forks source link

[Search v1] Implement receipt thumbnails #41629

Open luacmartins opened 2 weeks ago

luacmartins commented 2 weeks ago

We need to introduce receipt thumbnails in the search page as part of this section of the design doc

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~016b315a218e715b66
  • Upwork Job ID: 1788997846845743104
  • Last Price Increase: 2024-05-10
luacmartins commented 1 week ago

Holding PR is still in review

trjExpensify commented 1 week ago

You going to take this one?

luacmartins commented 1 week ago

Yes, I can work on it now that the holding PR is merged.

luacmartins commented 1 week ago

@Expensify/design could I get the new ReceiptPlus icon exported from Figma? Just wanna make sure I'm using the correct asset

dubielzyk-expensify commented 1 week ago

Here ya go

receipt-plus.svg.zip

luacmartins commented 1 week ago

Thanks @dubielzyk-expensify! For future reference, what are the steps to export these to make sure they are in line with our current icons?

dannymcclain commented 1 week ago

@luacmartins the steps are fairly convoluted for such a simple thing. Honestly it will likely be best to just ask a member of the design team to do it. But I will outline them anyways!

I know this seems pretty wacky for simply exporting a small UI icon, but we've found that Figma's exports aren't always reliable. We've found this process to consistently produce the best results. Design team, feel free to add any of your thoughts too! This is the way I do it though.

luacmartins commented 1 week ago

Thanks for listing out the steps. I think I'll keep asking you guys for icons haha

melvin-bot[bot] commented 1 week ago

Triggered auto assignment to @puneetlath (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

puneetlath commented 1 week ago

Will this be handled internally or externally?

luacmartins commented 1 week ago

I'm handling it and have PRs in review

melvin-bot[bot] commented 1 week ago

Job added to Upwork: https://www.upwork.com/jobs/~016b315a218e715b66

melvin-bot[bot] commented 1 week ago

Triggered auto assignment to Contributor Plus for review of internal employee PR - @shubham1206agra (Internal)

dubielzyk-expensify commented 6 days ago

Yep. Pretty much exactly what Danny said, but feel free to ping us and we'll whip it up for you @luacmartins