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

Added missing properties to IFile #445

Merged
merged 4 commits into from
May 17, 2021
Merged

Added missing properties to IFile #445

merged 4 commits into from
May 17, 2021

Conversation

DaleyKD
Copy link
Contributor

@DaleyKD DaleyKD commented May 12, 2021

I compared IFile to Microsoft.SharePoint.Client.File to see what properties were missing. So I added the following:

  • HasAlternateContentStreams
  • Length
  • ServerRedirectedUrl
  • VroomDriveID
  • VroomItemID

I don't know what HasAlternateContentStreams is, so I was unable to provide a comment for the property. I'm also not sure if I wrote the Tests properly.

…ength, ServerRedirectedUrl, VroomDriveID, and VroomItemID.
@DaleyKD DaleyKD mentioned this pull request May 12, 2021
1 task
@jansenbe
Copy link
Contributor

Looks great @DaleyKD ! I'll process it in the coming days (OOF currently). Really appreciate you're helping improve PnP Core SDK!

@DaleyKD
Copy link
Contributor Author

DaleyKD commented May 13, 2021

@jansenbe - Forgive me. I'm an absolute GitHub n00b (we use ADO TFS at work). I didn't realize doing another commit on my branch would add it to the PR. Honestly not sure what to do at this point.

jansenbe added a commit that referenced this pull request May 17, 2021
@jansenbe jansenbe merged commit 49da11d into pnp:dev May 17, 2021
@DaleyKD
Copy link
Contributor Author

DaleyKD commented May 17, 2021

@jansenbe - Thank you VERY much for fixing my test cases; I'll use this as a template for the future.

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