-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[$250] Unread chats are not shown bold but shown as unread when search #38704
Comments
Triggered auto assignment to @trjExpensify ( |
@dylanexpensify I'm heading out until the 2nd. Can I hand you this over to progress? I was wondering if it's related to #36075. Said differently, I don't know how this bug wasn't caught in the development of the fix for the other one really. |
@trjExpensify you got it! Will look into this today! Feel free to assign over! |
Ta! |
waiting for someone to ping in social lol |
@dylanexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
reviewing today! |
Having a hard time reproducing to be honest. Logging in takes me automatically to social. And I'm seeing LHN act according as it should. |
@m-natarajan can you still repro? |
Ahhh nice shout @iwiznia! I'll give it a try with that. |
I also reported similar here: https://expensify.slack.com/archives/C049HHMV9SM/p1713218931132899 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Ollyws ( |
Hello Expensify team! I'd like to make a proposal for this (my first!), but I'm trying to understand the issue fully. Please re-state the problem that we are trying to solve in this issue. What is the root cause of that problem? What changes do you think we should make in order to solve the problem? What alternative solutions did you explore? (Optional) Thanks for your consideration! (edited to use the proposal template provided) |
📣 @fitd-tech! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
So it seems to me that the problem here is that we intentionally don't bold new messages if the notifications are muted (#33779) but we only do this for @fitd-tech Thanks for the proposal, it's going in the right direction but I think we want to preserve the current behaviour for the LHN. |
Thanks @Ollyws! Let me know if I can do anything further. |
Feel free to update your proposal so that neither the LHN or search list are bolded when notifications are muted. |
ProposalPlease re-state the problem that we are trying to solve in this issue.The search list shows items with new activity in bold, even if they have been muted. The LHN does not show this activity, and we would like them to be consistent, so we should remove the bold styling from the search results when notifications are muted. (This description is the inverse of the original bug report, but seems more in line with our intent) What is the root cause of that problem?In the
In the App/src/pages/ChatFinderPage/index.tsx Lines 119 to 124 in db9be2c
What changes do you think we should make in order to solve the problem?I would like to add the same settings condition to the What alternative solutions did you explore? (Optional)I could add this check to the App/src/components/SelectionList/InviteMemberListItem.tsx Lines 97 to 107 in a310f77
...but since semantically we are setting |
@Ollyws can we get an update on the updated proposal? 🙇♂️ |
@fitd-tech's proposal LGTM, but we should get clarificaion from the internal engineer that this is absolutely the desired behaviour because as @fitd-tech mentioned it is kind of the inverse of the actual bug report. 🎀👀🎀 C+ reviewed |
Triggered auto assignment to @tylerkaraszewski, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Bump @tylerkaraszewski to review |
Issue not reproducible during KI retests. (First week) |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Issue not reproducible during KI retests. (Second week) |
@tylerkaraszewski, @Ollyws, @dylanexpensify Eep! 4 days overdue now. Issues have feelings too... |
@Ollyws can you repro? I can't |
@dylanexpensify I can still reproduce, the issue is that when we set the notification preference to |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Issue not reproducible during KI retests. (Third week) |
@tylerkaraszewski, @Ollyws, @dylanexpensify Eep! 4 days overdue now. Issues have feelings too... |
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:
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
Expensify/Expensify Issue URL:
Issue reported by: @iwiznia
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1710873829984469
Action Performed:
Expected Result:
#social public room should have unread indicator
Actual Result:
It does not show as unread
LHN is correct as there are many unread messages in the chat page.
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: