-
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
[$500] Workspace – WS name and avatar changed to default after deleting WS #38505
Comments
Triggered auto assignment to @alexpensify ( |
@alexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
We think that this bug might be related to #vip-vsp |
Adding this one to my testing list. |
No update yet |
I'll test this one soon. |
On my testing list |
Job added to Upwork: https://www.upwork.com/jobs/~01d7aa9ecfc6f965dd |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @alitoshmatov ( |
@alitoshmatov - we need to prevent the avatar from changing when a workspace is deleted. It looks like the other mention here does not appear in my recent tests. |
ProposalPlease re-state the problem that we are trying to solve in this issue.WS name and avatar change to default after deleting WS What is the root cause of that problem?We're clearing the workspace avatar here when deleting workspace. And after coming online, the workspace will be removed completely from Onyx. What changes do you think we should make in order to solve the problem?Since after coming online, the workspace will be removed completely from Onyx. We'll need to add a new field There might be similar back-end change needed to add support for the Optionallly, we can remove this line to preserve the workspace avatar in the Also we might want to do the same in related flow like What alternative solutions did you explore? (Optional)When retrieving the workspace avatar for display, for example in here, fallback to Another approach is we can modify the back-end to still return the deleted policy in the API response (and subsequent ReconnectApp API calls), so we still have the policy data in the front-end to show in archived reports, ... but we don't show or have any entry point to that deleted policy's pages in the front-end if the policy was deleted. |
No update yet, this one needs a proposal review. |
@alitoshmatov - please keep me posted if you cannot review this one and I'll reassign it. Thanks! |
@alexpensify Sorry, had some work on my plate, please reassign. |
I am happy to take over this one if @alitoshmatov is busy |
adding to hot picks and unassigning |
No update yet, this one is still a hot pick. |
@alexpensify Any update here? |
No update yet, it's still a hot pick |
Still a hot pick |
Hot Pick |
@alexpensify Should we continue on this issue anymore? It has been delayed for 5 months |
@DylanDylann, good call; I'm going to close now. |
@alexpensify Could you please handle payment in this issue? As mentioned on Slack, C and C+ should get fully paid because we are hired and did the work (The PR was created) cc @nkdengineer |
Thank you for flagging, my mistake here. Let me reopen and prepare the payment process tomorrow. |
Upwork job price has been updated to $259 |
Upwork job price has been updated to $250 |
Upwork job price has been updated to $500 |
Payouts due: 09-30-2024
Upwork job is here. This one falls under the $500 amount since it was opened before the pricing change. |
@nkdengineer - I sent an offer here: https://www.upwork.com/nx/wm/offer/104201713. Please accept and I can complete the payment process. Thanks! |
@alexpensify Thanks! I accepted however it seems the issue number in the offer is incorrect (#45879). If possible please update it to avoid confusion later. |
@nkdengineer thanks for flagging that my edits didn't save, I created a new offer here: https://www.upwork.com/nx/wm/offer/104222417 I'll withdraw the other one and please accept the new one. Sorry for the extra step but Upwork is blocking me from edits. |
@alexpensify Sure thing, I accepted |
Thanks and all set here. I've completed the payment process in Upwork. |
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.54-0
Reproducible in staging?: Y
Reproducible in production?: Y
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): applausetester+jp_e_category@applause.expensifail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Issue found when executing PR #38365
Action Performed:
Expected Result:
WS name and avatar don’t change to default after deleting WS
Actual Result:
WS name and avatar change to default after deleting WS
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6418213_1710787396452.WS_name_changed_to_default_after_deleting_WS.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @grgiaThe text was updated successfully, but these errors were encountered: