Closed lanitochka17 closed 1 month ago
Triggered auto assignment to @luacmartins (DeployBlockerCash
), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.
:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
We think that this bug might be related to #vip-vsp
@b4s36t4 @mollfpr this was found when testing this PR https://github.com/Expensify/App/pull/44702 any chance its regression from it?
Removing the blocker labels as this is super minor issue
This is fixed in the revert
https://github.com/user-attachments/assets/a8e11612-352e-48d9-a930-a6316f23abd2
https://github.com/user-attachments/assets/5fbc9aad-8750-482f-bbb7-fb82608303fb
@mountiny I tried the revert but it's not fix this issue.
Yea, @mollfpr I think our PR is not the offending PR for these issues.
Job added to Upwork: https://www.upwork.com/jobs/~012281ce39f663be9d
Gonna make this one external
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ishpaul777 (External
)
@b4s36t4 @mollfpr you are right, I have done one step differently and it caused this issue to not be reproducible 🤦
When creating a new group chat, the "Chat Report" header is briefly displayed in the group header before being replaced with the correct group name.
The root cause of this problem is that the code is incorrectly setting the group header to "Chat Report" before updating it to the correct group name.
To solve this problem, I propose updating the code to ensure that the correct group name is displayed in the group header immediately after creating the group, without displaying the "Chat Report" header.
Changes To implement this solution, we need to make the following changes:
Update the code that sets the group header to use the correct group name instead of "Chat Report". Ensure that the group name is displayed immediately after creating the group, without any delay or flickering. Solution Code
javascript
// Update the group header to use the correct group name
const groupName = getGroupName(); // Get the correct group name
setGroupHeader(groupName); // Set the group header to the correct group name
// Ensure the group name is displayed immediately after creating the group
useEffect(() => {
setGroupHeader(groupName); // Set the group header to the correct group name
}, [groupName]);
Expected Result The group header should correctly display the group name immediately after the group is created, without any temporary display of 'Chat report'.
Actual Result Group header 'Chat report' displayed briefly after creating a group.
I explored the following alternative solutions:
Delaying the display of the group header until the correct group name is available, but this may cause a brief delay in displaying the group header. Displaying a loading indicator or placeholder text in the group header until the correct group name is available, but this may not provide a seamless user experience.
Triggered auto assignment to @sonialiap (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.
When creating a new group chat, the "Chat Report" header is briefly displayed in the group header before being replaced with the correct group name.
The Report
action component seems to be opening chat report while report creation is in progress and is using defaults for faster creation while also using it for display which ends up briefly displaying wrong group name in header.
Provide group name and use default header i.e. Chat Report as a fallback value when opening Chat Report for the newly created group while report is also being created.
Update Report
component's openReport
to implement the above suggested fix.
After analyzing the reported issue, above suggested fix seems to work. The other alternative rather than using fallback would be to configure optimisticReport
correctly beforehand without overrides done for new report if opening a group chat.
📣 @dhruv-m-patel! 📣 Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork. Please follow these steps:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>
Contributor details Your Expensify account email: dhruvpatel.work@gmail.com Upwork Profile Link: https://www.upwork.com/freelancers/~0135a9ff376a6b5913
✅ Contributor details stored successfully. Thank you for contributing to Expensify!
I am not able to reproduce this issue only step that is missing is "After uploading, click on "View photo" which is removed in https://github.com/Expensify/App/pull/45863, Can this be retested if its still reproducable?
@lanitochka17
@sonialiap Can you please request a retest 🙇
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@sonialiap, @luacmartins, @ishpaul777 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
not overdue, waiting a retest to check if this is still reproducable..
@sonialiap, @luacmartins, @ishpaul777 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
@sonialiap @luacmartins @ishpaul777 this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
Applause reports this is not reproducible
I think we can close this issue then.
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
Yes i think this is good to close 👍
Closing. Thanks everyone!
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.10-2 Reproducible in staging?: Y Reproducible in production?: N If this was caught during regression testing, add the test name, ID and link from TestRail:N/A Email or phone of affected tester (no customers): shussain+chrome1@applausemail.com Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/44702
Action Performed:
Expected Result:
The group header should correctly display the group name immediately after the group is created, without any temporary display of 'Chat report'
Actual Result:
Group header 'Chat report' displayed briefly after creating a group
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
https://github.com/user-attachments/assets/17d82af3-5ecb-478d-93b5-9ef27396bd73
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @ishpaul777