-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Comments
Triggered auto assignment to @anmurali ( |
New Workspace
option from Global CreateNew Workspace
option from Global Create
Job added to Upwork: https://www.upwork.com/jobs/~01f04217489c09c36e |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @situchan ( |
ProposalPlease 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 App/src/pages/home/sidebar/SidebarScreen/FloatingActionButtonAndPopover.js Lines 193 to 204 in f697c89
What alternative solutions did you explore? (Optional)N/A |
ProposalPlease re-state the problem we are trying to solveRemove 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 |
ProposalPlease 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 App/src/pages/home/sidebar/SidebarScreen/FloatingActionButtonAndPopover.js Lines 209 to 220 in cbc4f1e
Plus we need delete Also I think about What alternative solutions did you explore? (Optional)NA |
Note The first comment from @GandalfGwaihir was just 'dibs' and proposal was edited in later. |
Triggered auto assignment to @flodnv, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
As this is new feature, this doesn't require specific root cause. They linked the correct code where to implement. |
New Workspace
option from Global CreateNew Workspace
option from Global Create
Upwork job price has been updated to $250 |
Decreasing the rewards due to relative simplicity of this job. |
Are we still doing this? |
I've put it in |
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. |
Cool, sounds good! |
Ideal Nav v2 was deployed to production. Is this still on hold? |
Yes, it is. |
Still on hold |
Still on hold |
Still on hold pending onboarding improvements. |
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. |
@JmillsExpensify accepted offer. Thanks |
C+ paid out, so we're all good here! |
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.Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: