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

[resources] inaccurate description for query parameters $top #18525

Closed
msyyc opened this issue Apr 2, 2022 · 0 comments · Fixed by #19063
Closed

[resources] inaccurate description for query parameters $top #18525

msyyc opened this issue Apr 2, 2022 · 0 comments · Fixed by #19063
Assignees
Labels
ARM bug This issue requires a change to an existing behavior in the product in order to be resolved. CXP Attention

Comments

@msyyc
Copy link
Member

msyyc commented Apr 2, 2022

"name": "$top",
"in": "query",
"required": false,
"type": "integer",
"format": "int32",
"description": "The number of results to return. If null is passed, returns all resources."

The description shows that it will return limited result if top is set. But actually, the call is pageable and service server will return limited result for every page, not that the final result is limited. So the description shall be like:

"name": "$top",
"in": "query",
"description": "The number of recommendations per page if a paged version of this API is being used.",
"required": false,
"type": "integer"

cc @weidongxu-microsoft

@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Apr 2, 2022
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label May 7, 2022
@RakeshMohanMSFT RakeshMohanMSFT self-assigned this May 13, 2022
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-net that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-net that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-java that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-resource-manager-schemas that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-powershell that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-go that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 16, 2022
{azure-rest-api-specs} | update description of $top parameter (Azure#19063)

Fixes Azure/azure-rest-api-specs#18525
FredericHeem pushed a commit to grucloud/azure-rest-api-specs that referenced this issue May 16, 2022
@RakeshMohanMSFT RakeshMohanMSFT added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ARM bug This issue requires a change to an existing behavior in the product in order to be resolved. CXP Attention
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants