-
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 2023-07-14] [$1000] The workspace logo disappears if changed the workspace picture on combined user/workspace chat #19905
Comments
Triggered auto assignment to @strepanier03 ( |
Bug0 Triage Checklist (Main S/O)
|
ProposalPlease re-state the problem that we are trying to solve in this issue.Workspace icon does not update in the workspace chat What is the root cause of that problem?The root cause of the issue is that we're not re-rendering the component once the What changes do you think we should make in order to solve the problem?Firstly, we need to pass Next, we'll need to load the
This will re-render the component and call We can optimise this approach further by just re-rendering when the avatar property of the attached policy changes. What alternative solutions did you explore? (Optional)None |
This should be fixed with #19750 so I'm going to put it on hold until that is merged so we can test again to be sure. |
PR has been deployed to Prod so I'm going to ask for someone to test again and then I'll action as necessary. |
I don't have a dev environment so I asked here if someone could test for me. |
Switching to Daily for now so I can remember to revisit this tomorrow. |
#19750 was fixed, but this issue still exists. I just tested it today again. error-2023-06-14_17.25.10.online-video-cutter.com.mp4 |
This is still happening on DEV so I'm going to remove the hold and push through the BZ process so we can get a re-evaluation of it. |
Job added to Upwork: https://www.upwork.com/jobs/~018e5f5da305633aff |
Triggered auto assignment to @tjferriss ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane ( |
Triggered auto assignment to @NikkiWines ( |
@rushatgabhane friendly bump here 🙇 |
Payments sent to @developerdavi and @priya-zha |
@rushatgabhane can you accept the offer in Upworks? |
|
Hi @tjferriss thanks for the offer on upwork but I'll be requesting payment on new dot. |
Made a manual request on new dot https://staging.new.expensify.com/r/4385861037826843 |
Easy, Melvin. We're just wrapping up payment. |
Found a regression from this issue's PR #22467 (comment). I need some assistance determine the correct solution. https://github.com/Expensify/App/pull/21657/files#r1275373499 |
@tjferriss Can you please summarize the appropriate individual payments for all parties involved in this issue? This is holding up @rushatgabhane's NewDot payments. More information on this compliance process in Slack. This may also be affected based on the possible regression above. |
The payment comment has been updated: #19905 (comment) |
@rushatgabhane Please respond to #19905 (comment) when you have time. Thanks. |
Reviewed details for @rushatgabhane. This is accurate based on summary from Business Reviewer and approved for payment in NewDot. |
@tjferriss, @NikkiWines, @rushatgabhane, @developerdavi Eep! 4 days overdue now. Issues have feelings too... |
@rushatgabhane bumping this comment once more #19905 (comment) |
@tjferriss i had responded to the in the PR https://github.com/Expensify/App/pull/21657/files#r1277398010 sorry for no updates on issue. we can close this i think |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
The workspace logo should have been updated to the picture set on the LHN in the combined user/workspace chat instead of making it go disappear
Actual Result:
The workspace logo disappears if changed the workspace picture on combined user/workspace chat
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.3.21-2
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: Any additional supporting documentation
chat-2023-05-29_11.26.13.mp4
Recording.825.mp4
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685339400060349
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: