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

Bug report: spo lisitem list / page size option docs mismatch #4927

Closed
dojcsakj opened this issue May 26, 2023 · 3 comments
Closed

Bug report: spo lisitem list / page size option docs mismatch #4927

dojcsakj opened this issue May 26, 2023 · 3 comments

Comments

@dojcsakj
Copy link
Contributor

Priority

(Low) Something is a little off

Description

m365 spo listitem list -h shows:

-p, --pageSize [pageSize]
Number of list items to return. Specify camlQuery or pageSize but not both. The default value is 5000.

But in fact short option for page size is: -s

Steps to reproduce

m365 spo listitem list -u <> -t <> -p 1

Expected results

Return only one list item

Actual results

Returns all list items

Diagnostics

No response

CLI for Microsoft 365 version

v6.7.0

nodejs version

v19.4.0

Operating system (environment)

Linux

Shell

bash

cli doctor

No response

Additional Info

It should work according help page or the help should be changed.

@milanholemans
Copy link
Contributor

Good catch @dojcsakj! Would you like to fix this one, or should we open it up for the community?

@dojcsakj
Copy link
Contributor Author

I'm happy to do that. :)

Should I modify the help text or the short option?

@milanholemans
Copy link
Contributor

I'm happy to do that. :)

Should I modify the help text or the short option?

Let's update the docs, else it would be a breaking change.
Thank you for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants