generated from future4code/labe-food-template
-
Notifications
You must be signed in to change notification settings - Fork 0
PR de Correção #24
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
Open
leticia-chijo
wants to merge
67
commits into
correcao-projeto
Choose a base branch
from
master
base: correcao-projeto
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
PR de Correção #24
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Projeto Base
Bibliotecas instaldas - Styled e Axios
estrutura base de pastas
profileUser e ProfileAddress
…s reutilivázeis para outros colegas
Subindo os arquivos para atualização e disponibilização de componente…
Login da pagina
atualização de login, endereço
Food allison
atualizações feita por Allison e Marcelo
atualização
Adicionando +1 rota
Tratando erros
Food allison
Food allison
atualização
leticia-chijo
commented
Mar 11, 2021
Author
leticia-chijo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oi gente! Esse projeto foi testado por dois colaboradores da Labenu que não são do frontend e eles deixaram alguns comentários sobre cada partezinha:
Pessoa 1:
**Login e Cadastro**
- Quando eu entro no aplicativo, não aparece a página inicial, falando o nome do aplicativo, apenas uma página vermelha e que some rapidamente.
- Todo o login funciona direitinho, apenas na hora do CPF, apanhei para colocar um CPF válido e foi só depois que eu coloquei os pontinhos e o tracinho, isso atrapalha bastante a navegabilidade do usuário. Minhca dica é, coloque esses pontinhos e tracinhos automaticamente e coloque um limite de caracteres (11 números), isso torna o login mais fácil!
**Busca e Seleção de Restaurantes**
- A busca e seleção de restaurante funciona perfeitamente, parabéns por isso!
- Eu só arrumaria o menuzinho embaixo, onde tem a casinha, o carrinho e a pessoinha. Quando você arrasta os restaurantes, alguns detalhizinhos aparecem atrás desse menuzinho, eu só colocaria ele encaixando perfeitamente na tela, sem que apareça os detalhes embaixo. Mas isso é só uma mini observação mesmo!
**Cardápio do Restaurante**
- O cardápio está funcionando direito, com todos os fluxos e botões funcionando (:
- A baixo vou falar algumas sugestões de coisas que eu colocaria, para tornar a página mais atrativa:
- A cor da descrição do restaurante está com um cinzinha bem clarinho, eu tornaria ele mais escuto e colocaria o nome do restaurante mais chamativo, talvez em negrito. Além disso, ainda na descrição do restaurante está o tempo do frete, depois Frete: R$ valor. Eu colocaria de outra forma, para deixar mais organizado as informações:
**"Endereço: XXXXXX
Tempo do frete : X minutos
Valor do frete: X reais"**
- Obs: eu vi que não está proposto no modelo para se inspirar, mas caso atualizem o projeto, eu mudaria isso!
- Agora nos itens para colocar no carrinho, eu colocaria o nome do intem mais chamativo també, talvez em negrito. Acaba chamando mais atenção e passa uma sensação de mais organizado.
- Grupo eu vou dar outra sugestão que está fora do modelo proposto (*desculpa sou aquariana e não sigo regras hahahha*). Quando a gente entra no restaurante e o seu cardápio, não aparece o menuzinho com a casinha, o carrinho e a pessoinha, eu colocaria ele também nessa parte do app, para facilitar a navegabilidade. Tipo sem ele, para eu acessar o carrinho, eu precisei entrar no menu principal e aí acessar o carrinho, tornando a navegabilidade menos fluída, então talvez colocar o menuzinho no cardápio pode ajudar nisso!
- Por fim aqui uns detalhizinhos para se arrumar:
- Bacio de Latte, a imagem do Doce "Solo especial - churros" não aparece
- Chun-li, a imagem da Bebida "Long Necks" não aparece e o Valor do "Rolinho - Chocolate com Morango" está fora do quadradinho.
**Carrinho e Finalizar Compra**
- Aqui eu também tornaria algumas palavras mais chamativa como "Endereço de Entrega". Também colocaria mais próxima as informações do restaurante e diferenciando o nome do restaurante com o restante dos dados, colocaria também "Rua" e "Tempo de frete" (de novo fora do modelo porposto, então são só sugestões que vão além do projeto)
- Quando eu finalizei um pedido, ele ainda permaneceu no carrinho, normalmente quando fazemos o pedido, ele acaba sendo excluindo, deduzindo que não tem mais nada no carrinho. Isso não aconteceu no app e acaba confundindo o usuário, "será que o pedido foi realizado? quando eu for fazer outra compra, isso vai ser incluído?". Então, eu limparia o carrinho assim que o pedido foi proposto.
- Por fim eu permitiria alterar a quantidade de produtos no próprio carrinho. Quando eu clico em remover, ele remove tudo, sugestão seria também colocar a opção de alterar a quantidade do item.
**Perfil, Editar Perfil e Histórico de Pedidos**
- Funciona tudo certinho! E está dentro do modelo proposto.
- Eu só colocaria algumas palavras em destaques (nome do restaurante, hostórico de pedidos) e usaria umas cores mais fortes, principalmente o cinza claro.
- Por fim mais uma sugestão que vai fora do modelo, quando tem as informações do meu perfil, colocaria:
- **"Nome: XXXX
Email: xxxx@xxx.com
CPF: XXXXXXXX"**
**Comentários Gerais Sobre o Projeto**
- Grupo parabéns pelo o projeto realizado, o app funciona direitinho, com todas as funcionalidades e fluxo dentro do esperado.
- Queridos e queridas, algumas das sugestões não estão dentro do modelo proposto (mal profs), são tudo coisas a mais mesmo e que eu acho que vai tornar o app mais incrível ainda!
- Parabéns pelo trabalhão devs de Frontend, agora bora brilhar no Backend e logo logo serem FullStacks aaaahehehe (:Pessoa 2:
**Login e Cadastro**
[APAGUE ISSO E ESCREVA AQUI]
Consegui me registrar! Ufa! Não sabia que o CPF válido precisava de "." "-". Digitei só números e passei um tempinho tentando entender o que é que estava errado. Isso porque o dígito
Outra sugestão, gente, é deixar claro que o "Clique aqui" que me levaria para a página de cadastro além do texto em si. Mudar a cor do texto, por exemplo, seria uma boa solução!
A imagem da home não carregou, mas é possível que tenha sido um problema da minha máquina...
De modo geral, essa parte ficou bem boa!
**Busca e Seleção de Restaurantes**
Tudo certinho aqui! Lista, filtragem e também os cardápios!
Seria bacana uma separação um pouco mais clara entre os 3 diferentes elementos dessa página: barra de busca, filtros e lista de restaurante. Não está ruim, mas se estivessem com uma cor de fundo diferente, sinto que ficaria um pouco mais claro.
**Cardápio do Restaurante**
Muito legal mesmo, gente! Tudo certinho por aqui também!
Sugestão: assim que eu selecionar quantas unidades quero adicionar ao carrinho, o popup poderia sumir direto. Isso porque ela já não teria mais utilidade depois de adicionar o item no carrinho.
Outra sugestão seria um botão de alterar a quantidade de itens que eu tenho no carrinho logo depois de adicionar esses itens. Eu remover tudo, mas não tirar somente uma das 3 porções de batata que pedi.
Por fim, gente, não consigo visualizar o menu inferior quando estou na página do restaurante.
**Carrinho e Finalizar Compra**
O frete não está sendo contabilizado na soma final do pedido.
Além disso, gente, assim que eu faço o pedido e o banner de "pedido em andamento" aparece, mudei de tela e fiquei confuso. "Será que eu cancelei o pedido?", "será que ainda tá em andamento?". Depois que saí da tela do carrinho, perdi a visiblidade do status do meu pedido.
**Perfil, Editar Perfil e Histórico de Pedidos**
[APAGUE ISSO E ESCREVA AQUI]
Consegui visualizar as minhas informações e editar elas! Que legal, gente :)
Uma sugestão: a possibilidade de editar somente um campo ao invés de ter que editar todos os 3. Tive que escrever o meu nome, o meu email e o CPF só para alterar um deles.
Outra sugestão seria ter as mesmas regras para input CPF tanto no cadastro quanto na edição do formulário. Acho que estão diferentes.
Só o histórico de pedido que, apesar de fazer a soma correta do valor do pedido, não me mostrou quais itens que eu comprei. Também veio com um número enorme que eu não entendi o que é, hahaha foi mal.
**Comentários Gerais Sobre o Projeto**
Ficou legal e bem intuitivo de usar! Parabéns, mesmo!
Todos os comentários que eu trouxe não mudam o fato de que as funcionalidades essenciais da plataforma. Por isso, parabéns!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR de Correção