-
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
[HOLD for payment 2024-07-22] [HOLD for payment 2024-07-17] [$250] Send invoice - "Chat Report" appears in room mention list after sending invoice #43988
Comments
Triggered auto assignment to @slafortune ( |
We think this issue might be related to the #vip-vsb |
ProposalPlease re-state the problem that we are trying to solve in this issue.A "Chat report" shown in the report mentions suggestions after sending an invoice. What is the root cause of that problem?A report will show in the suggestion if it passes the Lines 6934 to 6940 in e0ff35b
An invoice room is considered a chat room. Lines 968 to 970 in e0ff35b
The invoice room has a Chat report as the name. Chat report is basically a default name for a report. If you see the invoice room header, the name isn't Chat room because there is another logic to get the invoice room name. What changes do you think we should make in order to solve the problem?I don't think we should include the invoice room as a mentionable room because it's not a room that follows a room name condition that starts with a #. So, I suggest ignoring the invoice room in
|
Job added to Upwork: https://www.upwork.com/jobs/~01327604e093a2db66 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ahmedGaber93 ( |
@bernhardoj's proposal LGTM! 🎀 👀 🎀 C+ reviewed |
Triggered auto assignment to @iwiznia, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Hmmmm the proposal kind of makes sense as in I think will solve the issue, but the invoice room is kind of a chat room I think.... so not sure if it is technically correct. I see this was added here #40303 as part of the Invoicing v1 project. So let's ask @rezkiy37, @cristipaval and @madmax330 |
It wasn't mentioned in the doc. So we need to clarify with Exfy devs. |
The invoice room is a private workspace room. So, it makes much sense from the user standpoint to be mentionable. Is it possible to mention workspace chats? As part of the Invoicing v1 project, we want to make the invoice rooms behave as much as possible, such as workspace chats. cc'ing @davidcardoza to confirm. |
Not overdue, still discussing #43988 (comment) |
An invoice room mention should follow the same pattern as a workspace chat mention. There has been no proposal for an invoice room mention to deviate from the behavior of a workspace room. |
If we will allow the invoice room to be mentionable, what is the mentionable |
The mentionable For some background the
|
But a user display name can have multiple words and a room mention pattern is #{singleWord}. This mention only works for #admins, #announce, and user-created policy room. |
Interesting, let me bring this up in Slack. |
Discussing here - https://expensify.slack.com/archives/CSL3XBCCR/p1719536877926699 |
Not overdue, still Discussing |
PR is ready cc: @ahmedGaber93 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.5-13 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-07-17. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
I'll request in ND once payment is due. |
Ended contract for this GH with UpWorks |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.6-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-07-22. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Payment should be due tomorrow (07-17) |
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
Regression Test Proposal
Do we agree 👍 or 👎 |
Requested in ND. |
@slafortune Could you please pay out this when you have a chance. Thanks! |
@ahmedGaber93 Paid $250 for the C+ role via UpWorks @bernhardoj is owed $250 for the Contributor Role via NewDot |
$250 approved for @bernhardoj |
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: 1.4.85-4
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Expected Result:
If Chat Report is a valid room mention, selecting it from the list should result in highlighted #Chat Report in the composer.
Actual Result:
Selecting Chat Report from room mention list does not highlight the mention. # disappears.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
20240619_124104.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @slafortuneThe text was updated successfully, but these errors were encountered: