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

[HOLD] [$250] Remove New Workspace option from Global Create #36153

Closed
JmillsExpensify opened this issue Feb 8, 2024 · 32 comments
Closed

[HOLD] [$250] Remove New Workspace option from Global Create #36153

JmillsExpensify opened this issue Feb 8, 2024 · 32 comments
Assignees
Labels
External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Weekly KSv2

Comments

@JmillsExpensify
Copy link

JmillsExpensify commented Feb 8, 2024

As part of our ideal nav release, we have made it super easy to create a workspace, which was not previously the case. This includes an empty state to create a workspace when you tap the "wrench" in the bottom nav, as well as one click access to any workspaces you've previously created.

As a result, the New Workspace option is no longer needed. Let's remove it from Global Create.

image

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01f04217489c09c36e
  • Upwork Job ID: 1755604175468281856
  • Last Price Increase: 2024-02-08
  • Automatic offers:
    • situchan | Reviewer | 0
    • esh-g | Contributor | 0
Copy link

melvin-bot bot commented Feb 8, 2024

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Feb 8, 2024
@JmillsExpensify JmillsExpensify added External Added to denote the issue can be worked on by a contributor Daily KSv2 Help Wanted Apply this label when an issue is open to proposals by contributors and removed Weekly KSv2 labels Feb 8, 2024
@melvin-bot melvin-bot bot changed the title Remove New Workspace option from Global Create [$500] Remove New Workspace option from Global Create Feb 8, 2024
Copy link

melvin-bot bot commented Feb 8, 2024

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

Copy link

melvin-bot bot commented Feb 8, 2024

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

@allgandalf
Copy link
Contributor

allgandalf commented Feb 8, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

Remove New Workspace option from Global Create

What is the root cause of that problem?

Request feature

What changes do you think we should make in order to solve the problem?

Remove the New Worksapce option from
Remove from here

...(!props.isLoading && !Policy.hasActiveFreePolicy(props.allPolicies)
? [
{
displayInDefaultIconColor: true,
contentFit: 'contain',
icon: Expensicons.NewWorkspace,
iconWidth: 46,
iconHeight: 40,
text: translate('workspace.new.newWorkspace'),
description: translate('workspace.new.getTheExpensifyCardAndMore'),
onSelected: () => interceptAnonymousUser(() => App.createWorkspaceWithPolicyDraftAndNavigateToIt()),
},

What alternative solutions did you explore? (Optional)

N/A

@esh-g
Copy link
Contributor

esh-g commented Feb 8, 2024

Proposal

Please re-state the problem we are trying to solve

Remove New Workspace option from Global Create

What is the root cause of the problem?

N/A

What changes should be made to fix this?

We should remove the menu item here:

...(!props.isLoading && !Policy.hasActiveFreePolicy(props.allPolicies)
  ? [
        {
            displayInDefaultIconColor: true,
            contentFit: 'contain',
            icon: Expensicons.NewWorkspace,
            iconWidth: 46,
            iconHeight: 40,
            text: translate('workspace.new.newWorkspace'),
            description: translate('workspace.new.getTheExpensifyCardAndMore'),
            onSelected: () => interceptAnonymousUser(() => App.createWorkspaceWithPolicyDraftAndNavigateToIt()),
        },
    ]
  : []),

in FloatingActionButtonAndPopover

@ZhenjaHorbach
Copy link
Contributor

ZhenjaHorbach commented Feb 8, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

Remove New Workspace option from Global Create

What is the root cause of that problem?

New feature

What changes do you think we should make in order to solve the problem?

To fix this issue we can delete this element from FloatingActionButtonAndPopover

...(!props.isLoading && !Policy.hasActiveFreePolicy(props.allPolicies)
? [
{
displayInDefaultIconColor: true,
contentFit: 'contain',
icon: Expensicons.NewWorkspace,
iconWidth: 46,
iconHeight: 40,
text: translate('workspace.new.newWorkspace'),
description: translate('workspace.new.getTheExpensifyCardAndMore'),
onSelected: () => interceptAnonymousUser(() => App.createWorkspaceWithPolicyDraftAndNavigateToIt()),
},

Plus we need delete getTheExpensifyCardAndMore translate and Expensicons.NewWorkspace icon
Since this is the only place where we use these elements

Also I think about hasActiveFreePolicy
There are only 2 places where we use it
1st place here
And the second one is in deleteWorkspace
So we can move logic hasActiveFreePolicy inside deleteWorkspace to make the code cleaner

What alternative solutions did you explore? (Optional)

NA

@esh-g
Copy link
Contributor

esh-g commented Feb 8, 2024

Note

The first comment from @GandalfGwaihir was just 'dibs' and proposal was edited in later.

@situchan
Copy link
Contributor

situchan commented Feb 8, 2024

@esh-g was the first to provide complete proposal.
Code cleanup can be done in PR phase.

🎀 👀 🎀 C+ reviewed

Copy link

melvin-bot bot commented Feb 8, 2024

Triggered auto assignment to @flodnv, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@allgandalf
Copy link
Contributor

Hey @situchan , @esh-g too edited her proposal later on with complete proposal first she had only solution part:)

@ZhenjaHorbach
Copy link
Contributor

Actually I agree
I understand that this is a question of speed
But there isn’t even a full-fledged proposal with a link here

Screenshot 2024-02-08 at 15 59 22

@situchan
Copy link
Contributor

situchan commented Feb 8, 2024

As this is new feature, this doesn't require specific root cause. They linked the correct code where to implement.
@ZhenjaHorbach I think you already got benefit from other issue today. So let's end up unrelated discussion here.

@esh-g
Copy link
Contributor

esh-g commented Feb 8, 2024

As to address those concerns:

  • The first comment did indeed have a solution that could be understood rather than 'dibs'
  • It is allowed to edit proposals and if we compare the last edit time, my solution was still earlier
Screenshot 2024-02-08 at 8 32 52 PM Screenshot 2024-02-08 at 8 33 37 PM

I hope this clarifies my position, thanks for your understanding

@mountiny mountiny changed the title [$500] Remove New Workspace option from Global Create [$250] Remove New Workspace option from Global Create Feb 8, 2024
Copy link

melvin-bot bot commented Feb 8, 2024

Upwork job price has been updated to $250

@mountiny
Copy link
Contributor

mountiny commented Feb 8, 2024

Decreasing the rewards due to relative simplicity of this job.

@mountiny mountiny moved this to Release 1: Ideal Nav & Collect Simplfied Profile, Members, Categories, Workflows (approvals) in [#whatsnext] Wave 08 - Collect Plan Admins Feb 8, 2024
@melvin-bot melvin-bot bot added the Overdue label Feb 27, 2024
@trjExpensify
Copy link
Contributor

Are we still doing this?

@trjExpensify
Copy link
Contributor

I've put it in Polish for the time being.

@JmillsExpensify
Copy link
Author

I'm not sure if we will, though I think it makes sense to keep open for a little bit longer. Polish sounds right to me.

@melvin-bot melvin-bot bot removed the Overdue label Mar 6, 2024
@trjExpensify
Copy link
Contributor

Cool, sounds good!

@melvin-bot melvin-bot bot added the Overdue label Mar 14, 2024
@situchan
Copy link
Contributor

Ideal Nav v2 was deployed to production. Is this still on hold?

@JmillsExpensify
Copy link
Author

Yes, it is.

@melvin-bot melvin-bot bot removed the Overdue label Mar 27, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 5, 2024
@JmillsExpensify
Copy link
Author

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Apr 10, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 18, 2024
@JmillsExpensify
Copy link
Author

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Apr 24, 2024
@melvin-bot melvin-bot bot added the Overdue label May 2, 2024
@JmillsExpensify
Copy link
Author

Still on hold pending onboarding improvements.

@melvin-bot melvin-bot bot removed the Overdue label May 15, 2024
@JmillsExpensify
Copy link
Author

With no immediate plans to do this, I'm going to go ahead and close this issue. Additionally, since proposals were made and reviewed, I am willing to compensate the following:

@JmillsExpensify
Copy link
Author

Contributor paid out and C+ offer sent. Going to close this issue, but please comment here when you've accepted the offer and I'll issue payment.

@situchan
Copy link
Contributor

@JmillsExpensify accepted offer. Thanks

@JmillsExpensify
Copy link
Author

C+ paid out, so we're all good here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Weekly KSv2
Projects
No open projects
Archived in project
Status: Release 1: Ideal Nav & Collect Simplfied Profile, Members, Categories, Workflows (approvals)
Development

No branches or pull requests

9 participants