-
Notifications
You must be signed in to change notification settings - Fork 254
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
isModuleReady() com retorno incorreto #855
Comments
Olá @GuilhermeZety, ...
await Modular.isModuleReady<SeuModulo>();
// o que fazer depois do módulo estar pronto.
... Veja nessa parte da Doc: https://modular.flutterando.com.br/docs/flutter_modular/dependency-injection#asyncbind |
Opa! Então ele aguarda até o módulo estar pronto! okay! Mas e se eu quiser validar se um módulo está inicializado ou não? Pelos meus testes ele funciona pra isto também, visto que ele valida se o módulo ja está pronto... por essa explicação, caso o módulo não estivesse inicializado e ele ficasse esperando o módulo estar pronto ficaria esperando até cair no timeout(se tiver). Pelos meus testes caso eu passe um módulo que não foi inicializado o usecase dele retorna direto um Right(false) Não acho que seria muito problemático mudar este retorno para boolean, e retornar justo o que o usecase está retornando de fato! Ou talvez seria interessante mudar o nome da função para awaitModuleLoad() ou algo do tipo, pois o comum de algo como isAlgo() seria retornar um boolean |
@GuilhermeZety, não é possível ele retornar "diretamente" um true ou false, pois ele é um future e você teria que usar o await de qualquer forma. E se ele retornasse um Future<false>, não seria possível depois que o módulo tivesse pronto retornar um Future<true>, pois o retorno é único. |
Registro de Binds assincronos foram removidos por causarem problemas demais. |
Ao chamar o
Modular.isModuleReady<>()
o retorno declarado como um void.
porém o que deveria ser retornado neste caso é um bool com a info se está pronto ou não, como o próprio usecase ja retorna;
consegui retornar com um helper redefinindo o retorno da função, mas creio que ja deveria vir como padrão
The text was updated successfully, but these errors were encountered: