Skip to content

[Wave8] [Simplified Collect] Remove the free workspace menu items from collect workspace settings in NewDot #36606

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

Closed
trjExpensify opened this issue Feb 15, 2024 · 13 comments
Assignees
Labels
NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2

Comments

@trjExpensify
Copy link
Contributor

👋 We've started working on the PRs (like here and here) to implement the new workspace settings for collect. I noticed that we're showing the old free menu items in the LHN of the workspace settings along with the new menu items being added to collect workspaces only.

image

Let's remove these menu items from collect workspace settings only (leave free alone):

Cards
Reimbursements
Bills
Invoices
Travel
Bank account

Immediately then, we'll be left with:

Profile
Members

... and as we progress through implementation of release1 of wave8 in the coming weeks we'll see:

Profile
Members
Categories
Workflows

image

... ahead of then adding the rest and being conditional on being enabled in More features. Thanks! CC: @mountiny @luacmartins @lakchote @JmillsExpensify

@trjExpensify trjExpensify added Daily KSv2 NewFeature Something to build that is a new item. labels Feb 15, 2024
@trjExpensify trjExpensify self-assigned this Feb 15, 2024
Copy link

melvin-bot bot commented Feb 15, 2024

Current assignee @trjExpensify is eligible for the NewFeature assigner, not assigning anyone new.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Feb 15, 2024
@mountiny mountiny moved this to Release 1: Ideal Nav & Collect Simplfied Profile, Members, Categories, Workflows (approvals) in [#whatsnext] Wave 08 - Collect Plan Admins Feb 15, 2024
@mountiny mountiny self-assigned this Feb 15, 2024
@mountiny
Copy link
Contributor

cc @ArekChr this is kinda related to your PR too #36324

Would you be interested in handling this as its good to know the context and how it will fit the other pages? cc @rushatgabhane might be interested too

@ArekChr
Copy link
Contributor

ArekChr commented Feb 15, 2024

Sure @mountiny, I'm interested and happy to handle it

@mountiny
Copy link
Contributor

Thanks!

@rushatgabhane
Copy link
Member

@mountiny yes please! happy to help

@trjExpensify
Copy link
Contributor Author

Perfect, will be good for us to clear this hurdle for sure.

@lakchote
Copy link
Contributor

The logic to remove the free workspace menu items is already done in my PR.

You can see how I've enabled Workflows only for collect workspaces here.

@trjExpensify
Copy link
Contributor Author

Oh cool, so @ArekChr's PR is aligning with that for workflows yeah?

@trjExpensify
Copy link
Contributor Author

@lakchote confirming we're done here and close this out, right?

@trjExpensify trjExpensify moved this to Release 1: Spring 2024 (May) in [#whatsnext] #wave-collect Mar 5, 2024
@lakchote
Copy link
Contributor

lakchote commented Mar 8, 2024

@lakchote confirming we're done here and close this out, right?

Yes!

Closing the issue.

@lakchote lakchote closed this as completed Mar 8, 2024
@github-project-automation github-project-automation bot moved this from Release 1: Spring 2024 (May) to Done in [#whatsnext] #wave-collect Mar 8, 2024
@rushatgabhane
Copy link
Member

Hi @trjExpensify, could you please attach payment summary for this issue. Thanks 🙇

@trjExpensify
Copy link
Contributor Author

Ah, my bad. 😅 I didn't think there was an "independent" PR for this issue in the end. I see it now: #36672

Payment summary as follows:

$500 to @rushatgabhane for the C+ review.

Feel free to go ahead and request on NewDot :)

@JmillsExpensify
Copy link

$500 approved for @rushatgabhane based on summary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NewFeature Something to build that is a new item. Reviewing Has a PR in review 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

6 participants