Skip to content
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

Criar testes para erro 400 nos filtros (listagem de dataset e API) #460

Open
turicas opened this issue Oct 3, 2020 · 2 comments
Open

Criar testes para erro 400 nos filtros (listagem de dataset e API) #460

turicas opened this issue Oct 3, 2020 · 2 comments
Assignees

Comments

@turicas
Copy link
Owner

turicas commented Oct 3, 2020

8bbcf07

@berinhard
Copy link
Collaborator

Fechando essa issue pq o PR #465 já resolveu

@turicas
Copy link
Owner Author

turicas commented Nov 13, 2020

Fechando essa issue pq o PR #465 já resolveu

Vou reabrir para discutirmos aqui um caso de uso que, se implementado, ajudará ao usuário a detectar erros e trará mais aprendizado sobre o dataset. Se a pessoa enviar um filtro que não existe via query string, hoje o backend apenas ignora aquele parâmetro e devolve a página normalmente, exemplo: https://brasil.io/dataset/covid19/caso_full/?blablabla=123&state=SP é o mesmo que https://brasil.io/dataset/covid19/caso_full/?state=SP - acho que devemos mudar isso para gerar um erro 4xx caso algum parâmetro não seja reconhecido, assim a pessoa ficará sabendo que aquele parâmetro está incorreto e poderá detectar erros de digitação, por exemplo. O erro deve ser retornado também caso a pessoa passe um parâmetro que é um Field que não é frontend_filter (ou seja, um campo que ela não pode usar para filtrar).

@turicas turicas reopened this Nov 13, 2020
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

No branches or pull requests

2 participants