Skip to content

Conversation

@andyyngo
Copy link
Contributor

@andyyngo andyyngo commented Nov 27, 2025

Refactor error report to show all missing query parameters in one Exception
Close #122

@andyyngo andyyngo force-pushed the master branch 2 times, most recently from 6bd237b to 7b42ff0 Compare November 27, 2025 16:42
@sorndotdev sorndotdev linked an issue Nov 27, 2025 that may be closed by this pull request
@sorndotdev
Copy link
Collaborator

Should be #122 not #112

@andyyngo andyyngo force-pushed the master branch 2 times, most recently from eab61b3 to d101f88 Compare November 29, 2025 04:30
@andyyngo andyyngo requested a review from sorndotdev November 29, 2025 04:32
@andyyngo
Copy link
Contributor Author

@sorndotdev could you review this PR?

@andyyngo
Copy link
Contributor Author

Updated

Refactor error  report to show all  missing query parameters in one
Exception
@andyyngo
Copy link
Contributor Author

andyyngo commented Dec 1, 2025

@sorndotdev let's create an issue to update Spotless the styling guide catch these basic formatting. We can have a faster turn around when I can fix these issues on my own without going through your review.

@sorndotdev
Copy link
Collaborator

@andyyngo that sounds good - I agree

@sorndotdev sorndotdev merged commit ce2f5e8 into marketdataapi:master Dec 1, 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.

Refactor: Improve exception message for required query params

2 participants