Skip to content

Added CLI for M365 version for script "Export SharePoint List Data to CSV with attachments"#310

Merged
pkbullock merged 4 commits intopnp:mainfrom
nanddeepn:spo-export-sharepoint-list-items-to-csv
Jun 15, 2022
Merged

Added CLI for M365 version for script "Export SharePoint List Data to CSV with attachments"#310
pkbullock merged 4 commits intopnp:mainfrom
nanddeepn:spo-export-sharepoint-list-items-to-csv

Conversation

@nanddeepn
Copy link
Copy Markdown
Contributor

Added CLI for M365 version for script "Export SharePoint List Data to CSV with attachments"
Closes #275

@nanddeepn
Copy link
Copy Markdown
Contributor Author

nanddeepn commented Jun 10, 2022

Hi @Adam-it
Please review.

Issue:
The commands m365 spo listitem list with parameter --fields does not seem to work with lookup columns.

When lookup column is passed to --fields parameter, it throws below error:

Error: The query to field 'Parent' is not valid. The $select query string must specify the target fields and the $expand
 query string must contains Parent.

Also, same error occurs when m365 spo listitem get command is used with --properties parameter as lookup column

Can you please verify, if you experience the same.

Except the above issue, the PR works fine.

@Adam-it
Copy link
Copy Markdown
Member

Adam-it commented Jun 10, 2022

@nanddeepn your work is awesome 🤩. I will for sure have a look on this but on Monday as I am full with work over the weekend 😥, so please be patient 🙏.
as for the issues you described I think those are perfect for improvements in the CLI for M365 tool don't you think 🤔? I think we may open an issue referencing this script as example 😉. If you want I may add an issue or if you want to do it be my guest 😍. You may tag me in the issue if you want to engage me into the discussion 👍

@nanddeepn
Copy link
Copy Markdown
Contributor Author

Thank you @Adam-it
Have a nice weekend! Please take your time for review.

Meanwhile, I have created issue pnp/cli-microsoft365#3391 at pnp/cli-microsoft365 repo.

Copy link
Copy Markdown
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

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

pulled locally and tested
image

nothing to add
you rock 🤩

@pkbullock
Copy link
Copy Markdown
Member

Awesome sample - thank you @nanddeepn this should be live shortly. 😊🎉

@pkbullock pkbullock merged commit 5dcfeea into pnp:main Jun 15, 2022
@nanddeepn nanddeepn deleted the spo-export-sharepoint-list-items-to-csv branch June 17, 2022 06:54
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.

CLI for M365 version for script "Export SharePoint List Data to CSV with attachments"

3 participants