-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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.
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
[$1000] "----------New" message removed after two replies. #15064
Comments
Triggered auto assignment to @NicMendonca ( |
Bug0 Triage Checklist (Main S/O)
|
@kavimuru can you please fill out the platforms check box please? |
@NicMendonca I was not able to reproduce. Contributor was able to reproduce in all the platforms |
@kavimuru okay, thanks! The contributor can still reproduce: https://expensify.slack.com/archives/C049HHMV9SM/p1676470679632749?thread_ts=1675888386.030119&cid=C049HHMV9SM Trying on my end now! |
Another video from the contributor: https://expensify.slack.com/archives/C049HHMV9SM/p1676471724956199?thread_ts=1675888386.030119&cid=C049HHMV9SM |
Job added to Upwork: https://www.upwork.com/jobs/~01893843251fa388df |
Current assignee @NicMendonca is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @0xmiroslav ( |
Triggered auto assignment to @marcaaron ( |
This comment was marked as off-topic.
This comment was marked as off-topic.
I'm actually not following what the issue is here. @NicMendonca that sounds like the expected behavior to me. |
I'm gonna close because the repro steps are not really making sense to me. The new line clears when I switch from one web app to the other and focus the new one. I cannot leave "two replies" and get it to clear. It's just gone when switching. That seems a bit unexpected based on the behavior established in the unread indicators doc - but feels like something we should investigate separately. |
Can you please check this its not a bug?? |
Yes, the new marker should be removed after a new comment is left by the user who can see it. Can you please re-report this with the exact steps? It looks like you are sending a message from an email. If that's part of the reproduction please be very specific, thanks! |
Sir im receiving message from Web Chrome App on the video should i mention you at re-report post? |
Nope - just re-report please, thanks! |
we're centralizing discussion on #15212 since we're going to try and solve all this more holistically with a design doc |
This should be fixed! Can we please retest and close this issue |
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:
Expected Result
Green line "New" message should be remove after one reply
Actual Result:
"--New" message removed after two replies
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.67-2
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
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:
2023-02-09.14-59-40-1.mp4
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1675938702382709?thread_ts=1675888386.030119&cid=C049HHMV9SM
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: