Skip to content
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

Closed
1 of 6 tasks
lanitochka17 opened this issue Mar 18, 2024 · 59 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Hot Pick Ready for an engineer to pick up and run with Internal Requires API changes or must be handled by Expensify staff Monthly KSv2 Reviewing Has a PR in review

Comments

@lanitochka17
Copy link

lanitochka17 commented Mar 18, 2024

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:

  1. Log in
  2. Create new WS
  3. Change name and avatar
  4. Go to chats
  5. Go back to WS settings and delete WS
  6. Go back to chats and observe WS name and avatar

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?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

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
  • Upwork Job URL: https://www.upwork.com/jobs/~01d7aa9ecfc6f965dd
  • Upwork Job ID: 1773074259104903168
  • Last Price Increase: 2024-09-30
  • Automatic offers:
    • DylanDylann | Reviewer | 0
Issue OwnerCurrent Issue Owner: @grgia
@lanitochka17 lanitochka17 added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 18, 2024
Copy link

melvin-bot bot commented Mar 18, 2024

Triggered auto assignment to @alexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@lanitochka17
Copy link
Author

@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

@lanitochka17
Copy link
Author

We think that this bug might be related to #vip-vsp

@alexpensify
Copy link
Contributor

Adding this one to my testing list.

@alexpensify
Copy link
Contributor

No update yet

@alexpensify
Copy link
Contributor

I'll test this one soon.

@melvin-bot melvin-bot bot added the Overdue label Mar 25, 2024
@alexpensify
Copy link
Contributor

On my testing list

@melvin-bot melvin-bot bot removed the Overdue label Mar 26, 2024
@alexpensify
Copy link
Contributor

I tested and confirmed that the avatar changes when you delete the workspace. The name does not change.

Before

2024-03-27_12-42-47

After

2024-03-27_12-43-30

@alexpensify alexpensify added the External Added to denote the issue can be worked on by a contributor label Mar 27, 2024
@melvin-bot melvin-bot bot changed the title Workspace – WS name and avatar changed to default after deleting WS [$500] Workspace – WS name and avatar changed to default after deleting WS Mar 27, 2024
Copy link

melvin-bot bot commented Mar 27, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01d7aa9ecfc6f965dd

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Mar 27, 2024
Copy link

melvin-bot bot commented Mar 27, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @alitoshmatov (External)

@alexpensify
Copy link
Contributor

@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.

@nkdengineer
Copy link
Contributor

nkdengineer commented Mar 28, 2024

Proposal

Please 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 oldPolicyAvatar in the workspace-related reports, just like we do for workspace name here. Then when retrieving avatar for the report, we'll use oldPolicyAvatar of the report first if it exists (similar to here). This will make sure the archived report can still show the workspace avatar even after the workspace data itself was cleared. This is the same approach we currently use to preserve the policy name, that's why we can see the workspace name does not disappear, but the workspace avatar disappears.

There might be similar back-end change needed to add support for the oldPolicyAvatar field.

Optionallly, we can remove this line to preserve the workspace avatar in the policy in Onyx after deleting.

Also we might want to do the same in related flow like removing members, because the workspace data will also be deleted for that removed member.

What alternative solutions did you explore? (Optional)

When retrieving the workspace avatar for display, for example in here, fallback to avatarUrl first, if avatar is empty.

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.

@alexpensify
Copy link
Contributor

No update yet, this one needs a proposal review.

@melvin-bot melvin-bot bot added the Overdue label Apr 1, 2024
@alexpensify
Copy link
Contributor

@alitoshmatov - please keep me posted if you cannot review this one and I'll reassign it. Thanks!

@melvin-bot melvin-bot bot removed the Overdue label Apr 1, 2024
@alitoshmatov
Copy link
Contributor

@alexpensify Sorry, had some work on my plate, please reassign.

@DylanDylann
Copy link
Contributor

I am happy to take over this one if @alitoshmatov is busy

@alexpensify alexpensify added External Added to denote the issue can be worked on by a contributor and removed External Added to denote the issue can be worked on by a contributor labels Apr 3, 2024
@grgia grgia added Internal Requires API changes or must be handled by Expensify staff Hot Pick Ready for an engineer to pick up and run with and removed External Added to denote the issue can be worked on by a contributor labels Jun 19, 2024
@grgia grgia removed their assignment Jun 19, 2024
@grgia
Copy link
Contributor

grgia commented Jun 19, 2024

adding to hot picks and unassigning

@alexpensify
Copy link
Contributor

No update yet, this one is still a hot pick.

@DylanDylann
Copy link
Contributor

@alexpensify Any update here?

@alexpensify
Copy link
Contributor

alexpensify commented Aug 1, 2024

No update yet, it's still a hot pick

@alexpensify
Copy link
Contributor

Still a hot pick

@alexpensify
Copy link
Contributor

Hot Pick

@DylanDylann
Copy link
Contributor

@alexpensify Should we continue on this issue anymore? It has been delayed for 5 months

@alexpensify
Copy link
Contributor

@DylanDylann, good call; I'm going to close now.

@DylanDylann
Copy link
Contributor

DylanDylann commented Sep 26, 2024

@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

@alexpensify
Copy link
Contributor

Thank you for flagging, my mistake here. Let me reopen and prepare the payment process tomorrow.

@alexpensify alexpensify reopened this Sep 26, 2024
@alexpensify alexpensify changed the title [HOLD- BE] Workspace – WS name and avatar changed to default after deleting WS [$259] Workspace – WS name and avatar changed to default after deleting WS Sep 27, 2024
Copy link

melvin-bot bot commented Sep 27, 2024

Upwork job price has been updated to $259

@alexpensify alexpensify changed the title [$259] Workspace – WS name and avatar changed to default after deleting WS [$250] Workspace – WS name and avatar changed to default after deleting WS Sep 27, 2024
Copy link

melvin-bot bot commented Sep 27, 2024

Upwork job price has been updated to $250

@alexpensify alexpensify changed the title [$250] Workspace – WS name and avatar changed to default after deleting WS [$500] Workspace – WS name and avatar changed to default after deleting WS Sep 30, 2024
Copy link

melvin-bot bot commented Sep 30, 2024

Upwork job price has been updated to $500

@alexpensify
Copy link
Contributor

alexpensify commented Sep 30, 2024

Payouts due: 09-30-2024

Upwork job is here. This one falls under the $500 amount since it was opened before the pricing change.

@alexpensify
Copy link
Contributor

@nkdengineer - I sent an offer here: https://www.upwork.com/nx/wm/offer/104201713. Please accept and I can complete the payment process. Thanks!

@nkdengineer
Copy link
Contributor

@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.

@alexpensify
Copy link
Contributor

@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.

@nkdengineer
Copy link
Contributor

@alexpensify Sure thing, I accepted

@alexpensify
Copy link
Contributor

Thanks and all set here. I've completed the payment process in Upwork.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Hot Pick Ready for an engineer to pick up and run with Internal Requires API changes or must be handled by Expensify staff Monthly KSv2 Reviewing Has a PR in review
Projects
No open projects
Status: No status
Development

No branches or pull requests

6 participants