Skip to content

Feature/spo listitem get#465

Closed
mpowney wants to merge 9 commits intopnp:devfrom
mpowney:feature/spo-listitem-get
Closed

Feature/spo listitem get#465
mpowney wants to merge 9 commits intopnp:devfrom
mpowney:feature/spo-listitem-get

Conversation

@mpowney
Copy link
Copy Markdown
Contributor

@mpowney mpowney commented May 28, 2018

This PR adds a command to get list items as per #269

@coveralls
Copy link
Copy Markdown

coveralls commented May 28, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling f7e33da on mpowney:feature/spo-listitem-get into 2f23932 on SharePoint:dev.

@waldekmastykarz
Copy link
Copy Markdown
Member

Thanks @mpowney. Will have a look shortly!

@waldekmastykarz waldekmastykarz self-assigned this Jun 8, 2018
@waldekmastykarz
Copy link
Copy Markdown
Member

Hey @mpowney, when trying to retrieve a list item from a list using the listId option, I'm getting the following error:

Error: The expression "web/lists/(guid'a3815d57-23cb-4c5f-a0fb-455c43ae2a9d')/items(1)" is not valid.

Command:

o365 spo listitem get -u https://m365x526922.sharepoint.com/sites/team-a -i 1 --listId a3815d57-23cb-4c5f-a0fb-455c43ae2a9d -f Id,Title,PnPCity

Could you see if you can repro?

@mpowney mpowney force-pushed the feature/spo-listitem-get branch from 4ecb94e to 48b85fc Compare June 10, 2018 23:32
@mpowney
Copy link
Copy Markdown
Contributor Author

mpowney commented Jun 10, 2018

Hey @waldekmastykarz, I've reproduced the error

Error: The expression "web/lists/(guid'a3815d57-23cb-4c5f-a0fb-455c43ae2a9d')/items(1)" is not valid.

and fix is committed. Also re-based with dev.

@waldekmastykarz
Copy link
Copy Markdown
Member

Excellent! Thanks for the quick turnaround @mpowney!

@waldekmastykarz
Copy link
Copy Markdown
Member

Well done! Merged manually

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.

3 participants