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

[backlog] Potencial bug na inicialização do ambiente no MainboardStore #88

Open
shonorio opened this issue May 19, 2023 · 0 comments
Open

Comments

@shonorio
Copy link
Collaborator

A classe MainboardStore precisa inicializar a lista de pages que atualmente ocorre carregando uma das duas propriedades setupProgress ou pageController.

Esta abordagem tem os seguintes problemas:

  1. o setupProgress deveriam ser um action (método) e não uma property como está neste momento. Isto gera uma confusão com relação a utilização deste componente da classe. Ele é um property que retorna um void;
  2. o pageController é um property que executa um método para carregar o pages, só que isto ocorre em future sem tratamento, isto é uma má prática e deveria ser evitada no código;

Precisamos garantir que cada interface esteja realizando apenas a função que o nome atribui, e realizar os ajustes com relação a utilizar o future da maneira correta.

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

1 participant