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

Update set-pnpfield with no crawl to prevent the field from being returned in search results #3784

Closed
wants to merge 69 commits into from
Closed

Conversation

reshmee011
Copy link
Contributor

Type

  • New Feature

What is in this Pull Request ?

Add no crawl property to set-pnpfield Update set-pnpfield to prevent the field from being returned in search results

@gautamdsheth
Copy link
Collaborator

@reshmee011 - can you also please add it to docs as well ?

@reshmee011
Copy link
Contributor Author

@gautamdsheth : I have updated the docs. Forgot to push the changes for the docs earlier..

@gautamdsheth
Copy link
Collaborator

@reshmee011 - do we really need this ? I mean , we can easily do it via something like this.
Works for almost all properties.

Set-PnPField -Identity "Test" -Values @{NoCrawl=$true}

@reshmee011
Copy link
Contributor Author

@gautamdsheth , updating NoCrawl property is failing using method -values @{NoCrawl=$true} as well the switchparameter -nocrawl.

image

Something changed to prevent the update to the NoCrawl at column level. I will raise it with Microsoft and wait for an answer.

Happy for the PR to be closed and to use the -values option to update the property.

@gautamdsheth
Copy link
Collaborator

Ok, sure. Will keep it open, let me know what MS responds to this , we can merge it later then.
Thanks :)

erwinvanhunen and others added 11 commits February 24, 2024 02:35
Fixing Nullableboolean compile error due to the weird CSOM update
Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com>
* Update Update-PnPApp.md

* Update Unregister-PnPHubSite.md

* Update Unpublish-PnPApp.md

* Removed WhatIf parameter

* Update Test-PnPSite.md

Added Connection parameter

* Update Sync-PnPAppToTeams.md

* Added Connection parameter

* Update Test-PnPSite.md

* Update Stop-PnPFlowRun.md

* Update Set-PnpTeamsChannelUser.md

* Added parameter descriptions

---------

Co-authored-by: Gautam Sheth <gautamdsheth@outlook.com>
@KoenZomers KoenZomers added do-not-merge more info needed Further information is requested by user labels Mar 3, 2024
@reshmee011
Copy link
Contributor Author

It seems to be able to set the field as non crawlable, the "SharePoint Server Publishing Infrastructure" has to be activated which is not a recommended solution , hence closing the PR.

@reshmee011 reshmee011 closed this Apr 3, 2024
@reshmee011
Copy link
Contributor Author

It seems to be able to set the field as non crawlable, the "SharePoint Server Publishing Infrastructure" has to be activated which is not a recommended solution , hence closing the PR.

@reshmee011 reshmee011 deleted the permissions branch April 3, 2024 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge more info needed Further information is requested by user
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants