You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Got bit by this and documented it in alpacahq/marketstore#324. I feel that writing a test would be handy for tracking as well as maybe some protection client-side to avoid users hitting this.
2 questions:
should the client maybe do better error reporting from the server? Right now this is being deferred to user client code. I wonder if a flag like Client(raise_response_errors=True) would be handy?
There should be something in the docs stating that you can't use lower case structarray field names or even make Client.write() check for capital field names? This situation makes me think that the Client.create() approach (from Improvements #37) would allow for this without having to check in every .write() call.
The text was updated successfully, but these errors were encountered:
Got bit by this and documented it in alpacahq/marketstore#324. I feel that writing a test would be handy for tracking as well as maybe some protection client-side to avoid users hitting this.
2 questions:
Client(raise_response_errors=True)
would be handy?Client.write()
check for capital field names? This situation makes me think that theClient.create()
approach (from Improvements #37) would allow for this without having to check in every.write()
call.The text was updated successfully, but these errors were encountered: