-
Notifications
You must be signed in to change notification settings - Fork 324
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
Enhancement: add optional parameter to specify a folder in the app catalog for m365 spo app add
#6364
Comments
m365 spo app add
m365 spo app add
That's a great feature request. Thanks for submitting this @michaelmaillot! Looks good to go for me. @pnp/cli-for-microsoft-365-maintainers any further remarks before opening it up? |
Nice suggestion. Maybe for consistency, we should call it |
Before we do, let's double check if using folders in the app catalog is a supported scenario and we're not trying to promote something that's officially shouldn't be done (even though it's working). @VesaJuvonen do we officially support creating folders and uploading solutions to folders in the app catalog? |
@waldekmastykarz - that's a really good question. We have not really considered the scenario at this level, but technically that should work as file is still in the catalog and this will provide proper level of grouping between the solutions if there are a lot of solutions. Let me run this through with few people internally to avoid surprises. |
Thanks for your feedback @VesaJuvonen & @waldekmastykarz. TBH I actually met this scenario and I was the first surprised that it was actually possible. And the only thing that changes from "regular" usage of tenant app catalog is that if you set specific permissions on a folder, only people that have (read) access will be able to see apps inside, when they will browse through the org app store. But this won't impact installed ones (since compiled files are located in the |
Currently, SPFx solutions are deployed at the root of an app catalog, whether it's a tenant or a site collection one.
The idea is to add a new command option to specify a folder because in some organizations, there's only one tenant for the whole company, including the subsidiaries. In that case, permissions can be setup to not expose an app to everyone but only one subsidiary.
Property description idea:
--catalogFolderPath [catalogFolderPath]
The text was updated successfully, but these errors were encountered: