Skip to content

Add expiration date parameter to sharing link commands - #5489

Open
gautamdsheth wants to merge 1 commit into
devfrom
feat/1895-expiretime
Open

gautamdsheth wants to merge 1 commit into
devfrom
feat/1895-expiretime

Conversation

@gautamdsheth

@gautamdsheth gautamdsheth commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Type

  • Bug Fix
  • New Feature
  • Sample

Related Issues?

Fixes #1895

Depends on: pnp/pnpcore#1896

What is in this Pull Request ?

  • Added -ExpirationDateTime parameter to Add-PnPFileUserSharingLink and Add-PnPFolderUserSharingLink to allow links to expire after a specified date.
  • Updated documentation and examples for both commands to reflect the new parameter.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 20:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The assignments cannot compile until PnP.Core supports the property, and the PR description references the wrong issue.

3 open findings
What changed in this PR

Adds optional expiration dates to user-specific file and folder sharing links.

Changes:

  • Adds -ExpirationDateTime to both cmdlets.
  • Updates examples and parameter documentation.
  • Adds a changelog entry.
File Description
AddFolderUserSharingLink.cs Passes folder-link expiration options.
AddFileUserSharingLink.cs Passes file-link expiration options.
Add-PnPFolderUserSharingLink.md Documents folder-link expiration.
Add-PnPFileUserSharingLink.md Documents file-link expiration.
CHANGELOG.md Records the new parameter.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/Commands/Security/AddFileUserSharingLink.cs
Comment thread src/Commands/Security/AddFolderUserSharingLink.cs
Comment thread CHANGELOG.md

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants