carloea2 opened a new pull request, #8151: URL: https://github.com/apache/texera/pull/8151
### What changes were proposed in this PR? Reject dashboard sort values outside the documented `orderBy` grammar instead of silently removing query ordering. Keep all existing valid sort mappings unchanged. Before: an unknown sort value returned 200 with unordered pagination. After: an unknown sort value returns 400 with the rejected value. ### Any related issues, documentation, discussions? Closes #8147 ### How was this PR tested? - `sbt "WorkflowExecutionService / Test / testOnly org.apache.texera.web.resource.dashboard.DashboardResourceSpec"` - `sbt scalafmtCheckAll` - `sbt "scalafixAll --check"` - Built with `bin/local-dev.sh up texera-web --build --json` and ran the generated distribution on localhost. - Verified `orderBy=Bogus` returns 400 and `orderBy=NameAsc` still returns 200. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Codex -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
