-
Notifications
You must be signed in to change notification settings - Fork 197
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
Tell us what is missing in PnP Core SDK / what feature would you like to see added #897
Comments
Feature request: Add capability to easily add Quick Links webpart on a page, similar to Image or Text webpart. I would be happy to add a new cmdlet like - Add-PnPPageQuickLinksWebpart or something like that in PnP PowerShell. |
Feature request : |
with SharePointPnPCoreOnline I could connect Without a clientID
How can I achieve the above with the new pnp.core ? The new Minimal Console Example here requires registration of an AAD App and subsequent Client ID. thank you |
@joshmiller1FEI : the web login approach indeed is not in PnP Core SDK and that's by design, we're also not supporting Azure ACS anymore in PnP Core SDK. Both approaches are legacy authentication models, the only future proof authentication model for SharePoint is via OAuth and an Azure AD application. |
@jansenbe Azure AD authentication still doesn't have feature parity with ACS. For example:
|
Thanks for the feedback @fowl2. You're absolutely right in those, Regarding the need for certificates: yes, this is by design for calling SharePoint REST or CSOM APIs when using application permissions. If your application only uses Microsoft Graph calls, then a secret is sufficient. I'm not aware of any current plans to also allow a secret for calling SharePoint REST/CSOM using application permissions. |
Hi @jansenbe, In one of my past projects we somehow missed the ability to get terms by custom property, like in the good old TermSet.GetTermsWithCustomProperty(CustomPropertyMatchInformation) Method I played around with this implementation in this fork of PnP Core and it somehow works: https://github.com/mloitzl/pnpcore/tree/feature/termsbycustomproperty Do you think this has potential to get into the PnP Core Library? |
@mloitzl : I've taken a quick peek on the code and looks good. Can you please submit a PR? |
@gautamdsheth : with the new page API coming I don't see a need for that. |
@valpvt : can you elaborate a bit more on how you see this? |
Hi @jansenbe, it would be very interesting to include in pnp core the possibility to expand some fields during a list query (or if it exists, the documentation on how to use it 😄). Ex.: expand user field to get email/principal/sip... using rest API it's only one request but using pnp core we need to get the items and then create a batch to get the properties that we need. |
I just noticed an omission that you might want to implement for completions sake: When creating an |
Makes sense @rlv-dan , can you create a new issue for this so we can more easily track it. Thx! |
@jansenbe PnP Powershell relies on the hidden list at the admin site collection level and allows to get private channel sites. It seems like PnP Core SDK does not query this list because it does not retrieve private channel sites when using a service principal identity |
Feature request : We are able to use pnpcore to deploy SPFx app to app catalog. However, the app request permission to call Graph API. |
Thanks @larry-lau , that's a useful add. I've created a new issue to track this. |
Closing this issue as it's not an ideal way to deal with feature asks. Please just create a new issue if you feel something is missing |
To further grow PnP Core SDK we'd like to understand what features you'd like to see added to PnP Core SDK. If you've need a feature in your project or you've suggestions for features to add then please create a new issue in our issue list so that we can track your ask.
The text was updated successfully, but these errors were encountered: