Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[HOLD #16078][$2000] suggestion for Emoji remains even after sending message #15934

Closed
2 of 6 tasks
kavimuru opened this issue Mar 14, 2023 · 159 comments
Closed
2 of 6 tasks

[HOLD #16078][$2000] suggestion for Emoji remains even after sending message #15934

kavimuru opened this issue Mar 14, 2023 · 159 comments
Assignees
@s77rt @thiagobrez @miljakljajic @MonilBhavsar
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Improvement Item broken or needs improvement. retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause

Comments

@kavimuru
Copy link

kavimuru commented Mar 14, 2023 •
edited by melvin-bot bot
Loading

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

Action Performed:

  1. Go to any chat
  2. Type for any emiji ( do not select suggested emoji)
  3. Send message

Expected Result:

Suggested emoji should be removed after sending the message

Actual Result:

Emoiji suggestions shows

Workaround:

unknown

Platforms:

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

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: 1.2.92-0
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
Notes/Photos/Videos:

Screen_Recording_20230313_201639_Chrome.mp4

Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1678718820449989

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01eccf0dd16e54ab9d
  • Upwork Job ID: 1643225880592248832
  • Last Price Increase: 2023-04-24
@kavimuru kavimuru added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 14, 2023
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Mar 14, 2023
@MelvinBot
Copy link

Triggered auto assignment to @miljakljajic (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@MelvinBot
Copy link

MelvinBot commented Mar 14, 2023 •
edited by miljakljajic
Loading

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@MelvinBot
Copy link

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

@melvin-bot melvin-bot bot added the Overdue label Mar 17, 2023
@MelvinBot
Copy link

@miljakljajic Still overdue 6 days?! Let's take care of this!

@Expensify Expensify unlocked this conversation Mar 21, 2023
@miljakljajic
Copy link
Contributor

I am not able to reproduce this - I believe it may have been resolved while I was OOO.

@melvin-bot melvin-bot bot removed the Overdue label Mar 23, 2023
@tienifr
Copy link
Contributor

tienifr commented Mar 24, 2023 •
edited
Loading

@miljakljajic I still can reproduce on staging web

Screen.Recording.2023-03-24.at.09.55.03.mov

@tienifr
Copy link
Contributor

tienifr commented Mar 28, 2023

It seems @miljakljajic is not available during this time. @kavimuru Could you reproduce this issue?

@gadhiyamanan gadhiyamanan mentioned this issue Mar 30, 2023
Closed
6 tasks
@kadiealexander
Copy link
Contributor

This has also been reported here on mac web (chrome), but I was unable to reproduce it on v1.2.92-0 on the staging version. @gadhiyamanan, before this one gets reopened, could you please test on the app version v1.2.92-0 and confirm it's reproducible?

@gadhiyamanan
Copy link
Contributor

@kadiealexander I can reproduce it with the latest version v1.2.92-0

@Pujan92
Copy link
Contributor

Pujan92 commented Mar 30, 2023

This has also been reported here on mac web (chrome), but I was unable to reproduce it on v1.2.92-0 on the staging version. @gadhiyamanan, before this one gets reopened, could you please test on the app version v1.2.92-0 and confirm it's reproducible?

@kadiealexander assuming you might have used enter to send message but to reproduce you need to click on send button.

@jliexpensify
Copy link
Contributor

Ok, tested on Pixel 3a (v 92-0) and I can reproduce:

image

@jliexpensify
Copy link
Contributor

to reproduce you need to click on send button.

Yep, that's what I did - clicked on the button

@kadiealexander
Copy link
Contributor

kadiealexander commented Mar 30, 2023 •
edited
Loading

Thanks for that tip @Pujan92, I was able to repro on mac web chrome:

2023-03-30_16-36-32.mp4

@melvin-bot melvin-bot bot added the Overdue label Apr 3, 2023
@miljakljajic
Copy link
Contributor

Thank you for reproducing! Do we think this could be worked on externally? Adding engineering label to confirm

@melvin-bot melvin-bot bot removed the Overdue label Apr 3, 2023
@miljakljajic miljakljajic changed the title suggestions remains as it is even after sending message suggestion for Emoji remains even after sending message Apr 3, 2023
@MelvinBot
Copy link

Triggered auto assignment to @MonilBhavsar (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@s77rt
Copy link
Contributor

s77rt commented Mar 6, 2024

Same ^

@melvin-bot melvin-bot bot removed the Overdue label Mar 6, 2024
@melvin-bot melvin-bot bot added the Overdue label Mar 15, 2024
@s77rt
Copy link
Contributor

s77rt commented Mar 15, 2024

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Mar 15, 2024
@melvin-bot melvin-bot bot added the Overdue label Mar 25, 2024
@s77rt
Copy link
Contributor

s77rt commented Mar 25, 2024

Same ^

@melvin-bot melvin-bot bot removed the Overdue label Mar 25, 2024
@mallenexpensify mallenexpensify removed the External Added to denote the issue can be worked on by a contributor label Apr 3, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 3, 2024
@mallenexpensify mallenexpensify added External Added to denote the issue can be worked on by a contributor retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause and removed Overdue labels Apr 3, 2024
@melvin-bot melvin-bot bot added Overdue Help Wanted Apply this label when an issue is open to proposals by contributors labels Apr 3, 2024
@melvin-bot Melvin Bot
Copy link

melvin-bot bot commented Apr 3, 2024

Current assignee @s77rt is eligible for the External assigner, not assigning anyone new.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 Overdue labels Apr 3, 2024
@mallenexpensify
Copy link
Contributor

@miljakljajic there was no BZ here and you were the original assigned so adding you back. Threw the retest-weekly label here. If QA is able to reproduce, can you add this to the #vip-vsb project? Seems like it should live there. If not able to reproduce, plz close.

@melvin-bot melvin-bot bot added the Overdue label Apr 5, 2024
@s77rt
Copy link
Contributor

s77rt commented Apr 5, 2024

Still on hold

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Apr 5, 2024
@kavimuru
Copy link
Author

kavimuru commented Apr 8, 2024

Not reproducible anymore.

az_recorder_20240408_120303.mp4

@miljakljajic
Copy link
Contributor

Excellent, thank you! Closing.

@melvin-bot melvin-bot bot removed the Overdue label Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@s77rt s77rt

@thiagobrez thiagobrez

@miljakljajic miljakljajic

@MonilBhavsar MonilBhavsar

Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Improvement Item broken or needs improvement. retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests