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.56k stars 2.9k forks source link

MacOS - Report Action - No strike through and gray opacity is applied on offline deleted message #52753

Open IuliiaHerets opened 1 day ago

IuliiaHerets commented 1 day 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: v9.0.64-0 Reproducible in staging?: Y Reproducible in production?: N If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5238380 Email or phone of affected tester (no customers): expensify416+da2@gmail.com Issue reported by: Applause Internal Team

Action Performed:

  1. Navigate to any chat > Send message
  2. Apply offline mode
  3. Delete the sent message

Expected Result:

Message should be strike through, and grayed out

Actual Result:

Message is not strike through and grayed out.

Issue happens only on MacOS, not repro on WindowsOC.

Workaround:

Unknown

Platforms:

Screenshots/Videos

https://github.com/user-attachments/assets/332bf8c2-ea46-46a4-8829-a9ef11c9a02b

View all open jobs on GitHub

melvin-bot[bot] commented 1 day ago

Triggered auto assignment to @iwiznia (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

melvin-bot[bot] commented 1 day ago

💬 A slack conversation has been started in #expensify-open-source

melvin-bot[bot] commented 1 day ago

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

github-actions[bot] commented 1 day ago

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.
ChavdaSachin commented 1 day ago

PR https://github.com/Expensify/App/pull/52752#issue-2671757577 will fix this issue as well.

iwiznia commented 18 hours ago

PR was cherry picked @IuliiaHerets can you retest please?

IuliiaHerets commented 3 hours ago

@iwiznia Tester can't reproduce this issue in the build v9.0.64-2

https://github.com/user-attachments/assets/39784899-bdfd-4d58-a3d4-12dfaf4bdc1d