search

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
3.54k stars 2.88k forks source link

[$250] Chat summary email doesn't display video content #42301

Open m-natarajan opened 5 months ago

m-natarajan commented 5 months ago

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: 1.4.74-4 Reproducible in staging?: y Reproducible in production?: y 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: @dubielzyk-expensify Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1715820105101429

Action Performed:

  1. Open chat with someone
  2. Send video and have them not open or read the chat
  3. Observe recipients email

    Expected Result:

    The email summarizing "missed" chats should indicate that there's a video that was sent that's representative of the video

    Actual Result:

    It shows a stretched image that doesn't indicate that this was a video

    Workaround:

    unknown

    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 image (2)

Snip - NataNSMDejkq and Rajan - mnata utest@gmail com - Gmail - Google Chrome

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01e5cb10905ae74ed4
  • Upwork Job ID: 1793007823619047424
  • Last Price Increase: 2024-08-08
Issue OwnerCurrent Issue Owner: @ikevin127
melvin-bot[bot] commented 5 months ago

Triggered auto assignment to @dylanexpensify (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.

melvin-bot[bot] commented 5 months ago

@dylanexpensify Huh... This is 4 days overdue. Who can take care of this?

melvin-bot[bot] commented 5 months ago

Job added to Upwork: https://www.upwork.com/jobs/~01e5cb10905ae74ed4

dylanexpensify commented 5 months ago

Moving external

melvin-bot[bot] commented 5 months ago

Triggered auto assignment to Contributor-plus team member for initial proposal review - @ikevin127 (External)

samilabud commented 5 months ago

I think this is BE 🤔

ikevin127 commented 5 months ago

@dylanexpensify I think this should go Internal as am pretty sure the report email templating is handled from the BE side.

melvin-bot[bot] commented 5 months ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

melvin-bot[bot] commented 5 months ago

@ikevin127, @dylanexpensify Huh... This is 4 days overdue. Who can take care of this?

ikevin127 commented 5 months ago

Awaiting on decision for the issue to be labeled Internal.

dylanexpensify commented 5 months ago

Moved!

melvin-bot[bot] commented 5 months ago

@ikevin127 @dylanexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

melvin-bot[bot] commented 5 months ago

@ikevin127, @dylanexpensify Still overdue 6 days?! Let's take care of this!

ikevin127 commented 5 months ago

This went Internal since that's where the email templates are designed. @dylanexpensify Should I unassign myself or am I gonna be needed to test / review this once BE applies the fix ?

melvin-bot[bot] commented 5 months ago

@ikevin127, @dylanexpensify Huh... This is 4 days overdue. Who can take care of this?

ikevin127 commented 5 months ago

Internal: looking for a volunteer.

melvin-bot[bot] commented 4 months ago

@ikevin127, @dylanexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

ikevin127 commented 4 months ago

Internal: looking for a volunteer.

dylanexpensify commented 4 months ago

@ikevin127 you can stay assigned if you like, though might take a bit to get an internal volunteer

ikevin127 commented 4 months ago

Looking for a volunteer.

dylanexpensify commented 4 months ago

Similar

melvin-bot[bot] commented 4 months ago

@ikevin127, @dylanexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

ikevin127 commented 4 months ago

Looking for a volunteer 🎲

melvin-bot[bot] commented 4 months ago

@ikevin127, @dylanexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] commented 4 months ago

@ikevin127, @dylanexpensify Eep! 4 days overdue now. Issues have feelings too...

melvin-bot[bot] commented 4 months ago

@ikevin127, @dylanexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

ikevin127 commented 4 months ago

Still looking for a volunteer 🎲

melvin-bot[bot] commented 4 months ago

@ikevin127, @dylanexpensify Eep! 4 days overdue now. Issues have feelings too...

ikevin127 commented 4 months ago

Looking for a volunteer 🎲

dylanexpensify commented 4 months ago

Similar, let's move weekly

melvin-bot[bot] commented 3 months ago

This issue has not been updated in over 15 days. @ikevin127, @dylanexpensify eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

dylanexpensify commented 3 months ago

Same

melvin-bot[bot] commented 3 months ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

dylanexpensify commented 1 month ago

Same

dylanexpensify commented 1 month ago

this is a lower priority though

ikevin127 commented 1 month ago

Still looking for a volunteer 🎲

melvin-bot[bot] commented 3 weeks ago

Triggered auto assignment to @anmurali (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.

dylanexpensify commented 3 weeks ago

Reassigning as I head to parental leave next week! Thanks Anu!

melvin-bot[bot] commented 3 weeks ago

@anmurali, @ikevin127 Whoops! This issue is 2 days overdue. Let's get this updated quick!

ikevin127 commented 3 weeks ago

Still looking for a volunteer 🎲

melvin-bot[bot] commented 2 weeks ago

@anmurali, @ikevin127 Whoops! This issue is 2 days overdue. Let's get this updated quick!

ikevin127 commented 2 weeks ago

♻️ Issue is Internal - still looking for a volunteer!

@anmurali If not urgent, maybe this can be demoted to Weekly / Monthly to avoid daily Overdue.

melvin-bot[bot] commented 1 week ago

@anmurali, @ikevin127 Huh... This is 4 days overdue. Who can take care of this?

ikevin127 commented 1 week ago

♻️ Issue is Internal - still looking for a volunteer!

@anmurali If not urgent, maybe this can be demoted to Weekly / Monthly to avoid daily Overdue.

melvin-bot[bot] commented 1 week ago

@anmurali, @ikevin127 Whoops! This issue is 2 days overdue. Let's get this updated quick!

ikevin127 commented 1 week ago

♻️ Issue is Internal - still looking for a volunteer!

@anmurali If not urgent, maybe this can be demoted to Weekly / Monthly to avoid daily Overdue.

melvin-bot[bot] commented 4 days ago

@anmurali, @ikevin127 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

ikevin127 commented 4 days ago

♻️ Issue is Internal - still looking for a volunteer!

@anmurali If not urgent, maybe this can be demoted to Weekly / Monthly to avoid daily Overdue.