search

acesso-io / unico-check-flutter

Plugin Flutter que possibilita uso das SDK Android e IOS UnicoCheck
MIT License
2 stars 4 forks source link

Normalizando o projeto para o padrão de nullsfatey + effective dart #3

Closed Katekko closed 3 years ago

Katekko commented 3 years ago

Essa PR contempla a mudança do plugin para o novo padrão do dart (null safety). Com isso aproveitei para colocar algumas coisas no padrão do dart (effective dart)

Também foi adicionado a possibilidade de utilizar o FVM como versionador (eu só uso ele, então aproveitei e coloquei ele na PR também).

E por fim foi adicionado a lib de JSON SERIALIZABLE melhorando a legibilidade ao transformar alguma response em objeto dart.

Com isso existe a possibilidade de quebrar algumas coisas perante a regra de negócio do SDK, já que eu não tenho conhecimento profundo na regra.

Também pretendo trazer como sugestão a mudança de como o plugin trabalha (após eu me aprofundar na regra e processos do sdk), conseguindo padronizar ainda mais, melhorando a manutenção e a legibilidade.

Caso queiram discutir sobre algumas mudanças só comentar abaixo ou chamar em uma meeting. Pretendo utilizar esse meu fork para continuar o trabalho em nosso aplicativo, caso queiram discutir algumas mudanças para mantermos apenas um repositório, só falar.

Boa noite

MatheusDomingos commented 3 years ago

@Katekko primeiramente, show de bola sua pro atividade e visivelmente colocou o projeto em outro patamar. Queria bater um papo contigo via call referente as alterações realizadas. Voce consegue amanha, quinta feira (15/04) na parte da tarde?

Katekko commented 3 years ago

@Katekko primeiramente, show de bola sua pro atividade e visivelmente colocou o projeto em outro patamar. Queria bater um papo contigo via call referente as alterações realizadas. Voce consegue amanha, quinta feira (15/04) na parte da tarde?

Conseguimos as 17?

MatheusDomingos commented 3 years ago

Boa! Conseguimos sim, enviei o invite no email do seu github, confirma para nós! Grande abraço!

Katekko commented 3 years ago

Boa tarde @lucasselani, obrigado pelas considerações. As mudanças foram feitas e salvas na PR a frente dessa daqui. https://github.com/acesso-io/unico_check_flutter/pull/4 Creio que da para fechar essa PR e manter a discussão na outra.

@MatheusDomingos Peço perdão pelo atraso, houve um imprevisto aqui comigo e eu pedi a Mari para te avisar sobre, não sei se ela chegou a te avisar. Caso queira ainda fazer a meeting me de um toque.