Skip to content

Fix REST client query param serialization#5

Merged
JedimEmO merged 2 commits into
masterfrom
fix-rest-query-param-codegen
May 10, 2026
Merged

Fix REST client query param serialization#5
JedimEmO merged 2 commits into
masterfrom
fix-rest-query-param-codegen

Conversation

@JedimEmO
Copy link
Copy Markdown
Owner

@JedimEmO JedimEmO commented May 10, 2026

Summary

  • serialize generated REST client query params through the reqwest serde-backed query path instead of to_string()
  • emit repeated query keys for Vec and Option<Vec> query params
  • add e2e coverage for Vec query params and serde-renamed enum query params without Display
  • bump ras-rest-macro from 0.1.1 to 0.2.0 and add the Unreleased changelog entry

Fixes #3.

Verification

  • cargo test -p ras-rest-macro --all-features
  • cargo test -p ras-rest-macro --test e2e
  • cargo test -p ras-rest-macro --test http_integration
  • cargo check -p ras-rest-macro --all-features
  • cargo clippy -p ras-rest-macro --all-features --tests -- -D warnings
  • npm --prefix tests/playwright test

@JedimEmO JedimEmO merged commit edd2709 into master May 10, 2026
5 checks 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.

rest_service! client codegen: Vec<T> query params don't compile (and enums need Display)

1 participant