Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

Added Get-PnPFooter and Set-PnPFooter #2634

Merged
merged 8 commits into from
Jun 8, 2020
Merged

Added Get-PnPFooter and Set-PnPFooter #2634

merged 8 commits into from
Jun 8, 2020

Conversation

KoenZomers
Copy link
Collaborator

@KoenZomers KoenZomers commented Apr 15, 2020

Type

  • Bug Fix
  • New Feature
  • Sample

Related Issues?

N/A

What is in this Pull Request ?

Added Get-PnPFooter and Set-PnPFooter to work with the footer shown on Modern Communication pages

@KoenZomers KoenZomers added the status:checked and ready for merge ✔ PR has been checked and is ready for merge label Apr 15, 2020
@erwinvanhunen
Copy link
Member

Hej Koen, there is a 'rogue' file in this PR: SetHomePages - Copy.cs.

@erwinvanhunen erwinvanhunen removed the status:checked and ready for merge ✔ PR has been checked and is ready for merge label Jun 6, 2020
@KoenZomers
Copy link
Collaborator Author

KoenZomers commented Jun 8, 2020

Hej Koen, there is a 'rogue' file in this PR: SetHomePages - Copy.cs.

@erwinvanhunen Fixed and did some additional updates. Also prepared adding the ability to get and set the footer title and logo, but as that depends on a PnP Sites Core PR (pnp/PnP-Sites-Core#2673) which hasn't been merged yet, I have submitted those options in another PR (#2715). This PR is ready to be merged.

@KoenZomers KoenZomers added the status:checked and ready for merge ✔ PR has been checked and is ready for merge label Jun 8, 2020
@KoenZomers KoenZomers removed their assignment Jun 8, 2020
@erwinvanhunen erwinvanhunen merged commit be87cde into pnp:dev Jun 8, 2020
@Markus-Hanisch
Copy link
Contributor

How is the status of the Set-PnPFooter cmdlet? Is it already released? The reason why I am asking is, that I came across this SharePoint Site Footer > Controlling footer rendering using code or PowerShell where the Set-PnPFooter is already mentioned.

Connect-PnPOnline -Url "<SiteURL>" –Credentials (Get-Credential) 
Set-PnPFooter -Enabled:$false

However, I could not yet find any publicly available documentation on this cmdlet.

@mbratukha
Copy link

I can confirm that the version 3.22.2006.2 does not contain this method:
image

@KoenZomers
Copy link
Collaborator Author

That's odd, because it has been merged. Let me look into that.

@KoenZomers
Copy link
Collaborator Author

The code is definitely in the June 2020 release: 07637ed#diff-64eb1152df9e33a1a65c73794570ae34

Only thing I can think of is that you're also having On-Premises modules installed on your environment at which you are looking now. Only for SharePoint Online the Set-PnPFooter cmdlet is available.

Can you validate this by running Get-Module after connecting or trying Set-PnPFooter?

@KoenZomers KoenZomers deleted the AddSetPnPFooter branch June 22, 2020 12:44
@KoenZomers
Copy link
Collaborator Author

How is the status of the Set-PnPFooter cmdlet? Is it already released? The reason why I am asking is, that I came across this SharePoint Site Footer > Controlling footer rendering using code or PowerShell where the Set-PnPFooter is already mentioned.

Connect-PnPOnline -Url "<SiteURL>" –Credentials (Get-Credential) 
Set-PnPFooter -Enabled:$false

However, I could not yet find any publicly available documentation on this cmdlet.

Correct, I updated the public doc actually already before it was released in the June 2020 release. Not sure why it doesn't show in the PnP PowerShell documentation yet. Will look into that. The Set-PnPFooter is only available within the SharePointPnPPowerShellOnline module of June 2020. Doesn't matter if it's the first, intermediate 1 or intermediate 2 release. Please ensure you are using that.

@mbratukha
Copy link

I think it was PowerShell caching problem. I've upgraded PnP PowerShell Online right before executing these commands. It started to work after opening a new session of PowerShell.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:checked and ready for merge ✔ PR has been checked and is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants