Skip to content

Conversation

Clayton1805
Copy link
Owner

@Clayton1805 Clayton1805 commented Sep 7, 2024

New Pull Request Checklist

Issue Description

Closes: parse-community#2478

Approach

Para fazer essa implementação busquei aproveitar ao máximo a lógica já existente e também isolar ao máximo minha implementação para que ela não atrapalhe o fluxo já existente, tentar refatorar muitas coisas poderia ser bem perigoso visto que o código não possui testes unitários, então busquei entender bem o que estava alterando para não causar efeitos colaterais, a melhor solução que encontrei foi criar um novo estado que gerencia os campos de filtro vazios e só adicioná-los ao estado do filtro principal depois do campo da tabela ser selecionado pelo usuário, dessa forma eu consegui retirar a necessidade de que uma linha de filtro seja gerada com um campo já pré selecionado.

TODOs before merging

  • Add tests
  • Add changes to documentation (guides, repository pages, in-code descriptions)

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.

Improve filer dialog auto-complete usability

3 participants