Bruno Garcia
Bruno Garcia
Don't you have the error?
I use the REST Interface and I agree it should be improved. > in all endpoints: remove the path-like dot-separated format strings (e.g. .json) in favour of a query parameter...
@stickies-v, there is no risk for the user because there are these verifications, so covering them in the functional tests ensure if user types a negative or unknown value to...
@stickies-v, I understand your point but I believe that we should cover in the tests as much as possible, even if it is not something extremely critical. If there is...
> Code Review ACK [d3a434d](https://github.com/bitcoin/bitcoin/commit/d3a434d471da19e8e06ced10736b5efed78922d0) Perhaps a functional test for this new parameter can be added. Functional test for it would be great, will work on it.
> Thanks, @achow101. I agree with you, the better approach would be add the option to the options object. Going to change it, thanks again.
force-pushed addressing comments from @achow101
Hi, @S3RK. Thanks for the comments, I didn't review #23367 yet, hope to do it soon. > When you force a solution without change you forego a more economically efficient...
Force-pushed rebasing and addressing last @achow101's comment.