[$250] mWeb - Room - Whisper is not shown for mention added while edit comment

[lanitochka17]

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: 9.0.7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4724888
Issue reported by: Applause - Internal Team

Action Performed:

1. Go to https://staging.new.expensify.com/home
2. Tap on a room report
3. Enter jaihanumanblog@gmail.com
   Only bind a single Keyboard.addListener('keyboardDidShow') event #567 (email id and random room mention)
4. Send the message
5. Note whisper message is shown
6. Long press the comment and select edit comment
7. Enter jaihanumanblog+6788@gmail.com
   [CP Staging] revert 56316 #56778 (email id and random room mention)
8. Send the message

Expected Result:

Whisper must be shown for mention added while edit comment

Actual Result:

Whisper is not shown for mention added while edit comment

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6543563_1721114547362.Sc._bh.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01490add6334f2b67a
• Upwork Job ID: 1813353966729699916
• Last Price Increase: 2024-07-16

Issue Owner

Current Issue Owner: @marcochavezf

[melvin-bot]

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

[lanitochka17]

@JmillsExpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[lanitochka17]

We think that this bug might be related to #vip-vsp

[JmillsExpensify]

Yes, that's correct. Opening up to the community.

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~01490add6334f2b67a

[melvin-bot]

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

[Tony-MK]

@JmillsExpensify and @suneox, I believe this issue should handled in the back end because the updateComment response does not include the "ACTIONABLEREPORTMENTIONWHISPER" action like the AddComment response.

Could we check this in the back end?

Thanks

[suneox]

@JmillsExpensify and @suneox, I believe this issue should handled in the back end because the updateComment response does include the "ACTIONABLEREPORTMENTIONWHISPER" action like the AddComment response.

@Tony-MK Thank you for your investigate, I'll be check in few hours

[suneox]

I have double-checked that the updateComment API doesn’t update the message content for actionName is ACTIONABLEREPORTMENTIONWHISPER. Therefore, I think this issue should be moved to internal.

🎀 👀 🎀 C+ reviewed

[melvin-bot]

Triggered auto assignment to @marcochavezf, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[suneox]

Not overdue, still waiting for an internal fix

[marcochavezf]

No update, I will move to weekly since it's vip-vsb and low

[melvin-bot]

@JmillsExpensify @marcochavezf @suneox 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!

[marcochavezf]

No update, addressing other high priority items this week

[marcochavezf]

I will check it out today or tomorrow, I will move it to daily meanwhile

[melvin-bot]

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

[marcochavezf]

Sorry, no update. I've been caught up with other priorities.

[suneox]

This issue will be handled by the internal team, so I’d like to self-unassign

[marcochavezf]

Moving it back to weekly

[marcochavezf]

No update, addressing other higher priority issues

[marcochavezf]

No update

[marcochavezf]

Not a priority, unassigning myself for now

[JmillsExpensify]

Just going to close if this isn't a priority.