Skip to content

Conversation

@anasik
Copy link
Contributor

@anasik anasik commented Jul 29, 2025

Also, fixes #1505 but for 9.x

@anasik anasik changed the title fixed NoDollar top and skip in nextlink Fixes #1505 for 9.x: NoDollar top and skip being copied to @odata.nextLink Jul 29, 2025
@anasik anasik force-pushed the noDollarNextLinkFIxMain branch from 99d6bc2 to 757428e Compare July 30, 2025 15:38
@anasik anasik requested a review from xuzhg July 31, 2025 14:23
xuzhg
xuzhg previously approved these changes Jul 31, 2025
@xuzhg
Copy link
Member

xuzhg commented Jul 31, 2025

/AzurePipelines run OData-AspNetCoreOData-main-rolling-1ES

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@anasik
Copy link
Contributor Author

anasik commented Aug 1, 2025

@xuzhg, same test failing here too?

@xuzhg
Copy link
Member

xuzhg commented Aug 1, 2025

@xuzhg, same test failing here too?

I didn't check this, but I think it should be the same. Please run those test cases at your side to verify.

@xuzhg xuzhg merged commit c89d76e into OData:main Aug 3, 2025
1 check passed
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.

"?top" and "?skip" unmodified in nextLink when EnableNoDollarQueryOptions is true.

2 participants