cezar-pereira / store

0 stars 0 forks source link

Feedback vesti #1

Open klausgreiner opened 2 years ago

klausgreiner commented 2 years ago

Muito obrigado pela participação no processos estamos avaliando todos os candidatos, devemos entrar em contato logo para marcar segunda parte da entrevista técnica. Qualquer informação ou mudança eu entro em contato pelo Whatsapp. Muito obrigado pela dedicação e espero falar contigo logo!

Observações gerais sobre o apk entregue, estas envolvem bugs encontrados, melhorias de ux. APK ENTREGUE ESTAVA QUEBRADO mensagem no-connection-internet

Positivos: -Aplicativo funciona como esperado; -Nenhum bug encontrado; Negativos:

Observações técnicas (Sugestões de melhoria ou estudo ou problemas encontrados debuggando o app):

cezar-pereira commented 2 years ago

Obrigado pelo feedback.

mensagem no-connection-internet

Erro meu, esqueci de colocar a permissão de internet, então por isso deu essa tratativa de sem conexão. Mas isso mostra que estou tratando 😅 Falha ao seguir design da bottom bar; Tem razão, acabei esquecendo desse componente. Seria interessante se houvesse uma maneira de remover filtro ao clicar novamente na categoria (new arrivals mostra todos); Mas está fazendo isso, ao clicar em new arrivals ele mostra todos os produtos. Maneira de exibir quantidade de itens e posição do carrinho não são chamativos e quando itens são selecionados eles escondem parte do icone; Não entendi muito bem… mas a questão do carrinho foi um “plus” meu. -Exception The ParentDataWidget Expanded(flex: 1) wants to apply ParentData of type FlexParentData to a RenderObject, which has been set up to accept ParentData of incompatible type ParentData :(. Não observei esse erro. -Utilização de setState(), pq não utilizou observables e actions? :/ ; Desculpa, mas não acho setState um problema, principalmente se utilizado no contexto de um componente… Tens razão, em alguns métodos deixei de usar o @action, mas sempre usei o @observable.

Em 6 de abr. de 2022, à(s) 11:31, Klaus Greiner Fiorini @.***> escreveu:

Muito obrigado pela participação no processos estamos avaliando todos os candidatos, devemos entrar em contato logo para marcar segunda parte da entrevista técnica. Qualquer informação ou mudança eu entro em contato pelo Whatsapp. Muito obrigado pela dedicação e espero falar contigo logo!

Observações gerais sobre o apk entregue, estas envolvem bugs encontrados, melhorias de ux. APK ENTREGUE ESTAVA QUEBRADO mensagem no-connection-internet

Positivos: -Aplicativo funciona como esperado; -Nenhum bug encontrado; Negativos:

Falha ao seguir design da bottom bar; Seria interessante se houvesse uma maneira de remover filtro ao clicar novamente na categoria (new arrivals mostra todos); Maneira de exibir quantidade de itens e posição do carrinho não são chamativos e quando itens são selecionados eles escondem parte do icone; Observações técnicas (Sugestões de melhoria ou estudo ou problemas encontrados debuggando o app):

Utilização de const files e criação de temas :) ; -Exception The ParentDataWidget Expanded(flex: 1) wants to apply ParentData of type FlexParentData to a RenderObject, which has been set up to accept ParentData of incompatible type ParentData :(. -Criação de testes em múltiplas camadas e utilização de mocks :); -Utilização de setState(), pq não utilizou observables e actions? :/ ; -Interessante utilização de Dartz para lidar com fluxos de exception :) ; — Reply to this email directly, view it on GitHub https://github.com/cezar-pereira/store/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALPNU3VJYFUJDVUFJR4J6VDVDWN4LANCNFSM5SWHQD2A. You are receiving this because you are subscribed to this thread.