-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[API] Improve error messages #25
Comments
Seria legal mandar o status code nos erros de API, pois muitos clientes de alto nível interpretam essa informação no consumo do retorno da chamada REST. Uma sugestão que levantei junto com o Lycan seria BadRequest para datas futuras e Not Found para datas anteriores a 1/1/2003. Status 5xx seria legal para quando houver erros do Crawler também! |
@wbruno bora terminar essas validações? |
yep |
+1 |
Caros, para vosso conhecimendo, estou construindo um cliente para Android da API como side project. Comecei semana passada para brincar com algumas coisas do meu interesse, está ficando bem legal, além da consulta dos dados por dia, no detalhe de cada represa vou fazer um gráfico comparado (hoje x 6 meses x 1 ano, 3 requests para montar). Esses erros tratados serão muito bem-vindos! :) |
Yeap. Pode deixar @ubiratansoares vamos arrumar isso ai sim e ainda mais, vamos implementar testes =) Quanto a sua necessidade, acredito que a issue #22 vai vir bem a calhar certo? Att |
@rafaell-lycan Achei #22 animal, mas se vierem todos os dias no range de um ano, vai ser um payload monstro na volta e talvez d+ pra mim, rsrsrs. A princípio, tô pensando só em 3 datas mesmo, se vc estiver voltando as pontas apenas do range, daí eu consigo resolver com 2 GETs Agora, se além de range, no futuro tiver uma consulta do tipo /dates?values=d1,d2,d3 fica bala pra mim pq eu não preciso fazer 3+ requests pra montar o gráfico que estou querendo!!. BTW .... |
When some happening and the code are 40X, 50X, we need to send a message with error_message and maybe some code.
The text was updated successfully, but these errors were encountered: