-
Notifications
You must be signed in to change notification settings - Fork 432
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
sharepoint listitem expand=fields #436
Comments
I'm not into this for the moment. If you want PR are welcome |
It's only a few lines to add this to the code as a parameter for the
function, but I'm unable to push my code update due to permissions.
…On Mon, May 4, 2020 at 5:53 AM Alejandro Casanovas ***@***.***> wrote:
I'm not into this for the moment. If you want PR are welcome
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#436 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA6PBVWFG5X2KRGMJSABD63RP2NELANCNFSM4MVV6GFA>
.
|
You can fork and make a Pr without problems. I can't help you beyond that |
I’ll handle it on my own…
I was just trying to help you and your project is all.
… On May 4, 2020, at 11:44 AM, Alejandro Casanovas ***@***.***> wrote:
You can fork and make a Pr without problems. I can't help you beyond that
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
, or unsubscribe
.<embedded image>
|
@techon, you would be helping if you created a PR like he mentioned - it is customary to fork repositories you don't own.. Anyways, I ran in to this issue as well and went ahead and submitted a PR for you @janscas. Please review/make changes as necessary. Wasn't sure how you wanted to handle an invalid field name (I used log.warning) |
@kwollaston thank you! |
Add support for the URL parameter ?expand=fields
SharepointList.get_items()
Or is there another way to pull any additional fields?
The text was updated successfully, but these errors were encountered: