Skip to content

Conversation

@jbrichau
Copy link
Member

Add optional query parameters to the routing definitions in Seaside REST.

Before this possibility, any query parameter that can be present in the request MUST be in some route declaration's path specification. When multiple optional query parameters are possible for the same path, it becomes very impractical to specify all possible path specifications.

This PR introduces optional query parameters.

@codecov
Copy link

codecov bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 87.50000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.92%. Comparing base (1935560) to head (6b67dff).
⚠️ Report is 26 commits behind head on master.

Files with missing lines Patch % Lines
...Builder.class/instance/optionalQueryParameters..st 0.00% 5 Missing ⚠️
...agmaStubHandler.class/instance/getAllDocuments..st 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1486      +/-   ##
==========================================
+ Coverage   51.90%   51.92%   +0.01%     
==========================================
  Files        9175     9178       +3     
  Lines       76836    76874      +38     
==========================================
+ Hits        39884    39916      +32     
- Misses      36952    36958       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants