Skip to content

PR de Correção#18

Open
Janaylla wants to merge 42 commits intocorrecao-projetofrom
master
Open

PR de Correção#18
Janaylla wants to merge 42 commits intocorrecao-projetofrom
master

Conversation

@Janaylla
Copy link
Copy Markdown

@Janaylla Janaylla commented Oct 4, 2021

Parabéns pela entrega

Excelente entrega, o trabalho foi muito bem desenvolvido e esta lindo.
Observações:

  • Foram usadas 5 variáveis no state para trocar a pagina, apenas uma string seria suficiente para fazer essa mudança.
    paginaHome: true,
    paginaAnunciar: false,
    paginaPesquisar: false,
    paginaDetalhe: false,
    paginaCarrinho: false,
  • As variáveis do inputs controlados da pesquisa poderia ter sido criadas no component pesquisa e passar os valores por argumento das funções para para o AppConteiner. As funções de ordenação poderiam ser unificadas usando if/else e passando a opção de ordenação por argumento. Desta forma o código ficaria mais limpo e mais legível.
 <Pesquisar
          onClickDetalhe={this.onClickDetalhe}
          allJobs={this.state.allJobs}
          onClickPrecoCrescente={this.onClickPrecoCrescente}
          onClickPrecoDecrescente={this.onClickPrecoDecrescente}
          onClickNomeCrescente={this.onClickNomeCrescente}
          onClickNomeDecrescente={this.onClickNomeDecrescente}
          onClickPrazoCrescente={this.onClickPrazoCrescente}
          onClickPrazoDecrescente={this.onClickPrazoDecrescente}
          filtroValorMaximo={this.state.filtroValorMaximo}
          filtroValorMinimo={this.state.filtroValorMinimo}
          filtroNome={this.state.filtroNome}
          onChangeInputValorMaximo={this.onChangeInputValorMaximo}
          onChangeInputValorMinimo={this.onChangeInputValorMinimo}
          onChangeInputNome={this.onChangeInputNome}
          filtrarPorValor={this.filtrarPorValor}
          filtrarPorNome={this.filtrarPorNome}
          getAllJobs={this.getAllJobs}
        />

Aplicando a sugestão:

 <Pesquisar
          onClickDetalhe={this.onClickDetalhe}
          allJobs={this.state.allJobs}
          onClickOrdenacao={this.onClickOrdenacao}
          filtroValorMaximo={this.state.filtroValorMaximo}
          filtroValorMinimo={this.state.filtroValorMinimo}
          filtroNome={this.state.filtroNome}
          filtrarPorValor={this.filtrarPorValor}
          filtrarPorNome={this.filtrarPorNome}
          getAllJobs={this.getAllJobs}
        />
  • Seria interessante que o botão de adicionar carrinho pudesse ser clicado na lista de serviços e não somente nos detalhes de cada produto.

silviordjr and others added 30 commits September 28, 2021 15:40
Labeninjas - primeiras funcionalidades
Cadastro de serviços e ordenação
…geanunciar

responsividade da pagina cadastrar
Responsividade Carrinho e Detalhe do produto
Copy link
Copy Markdown
Contributor

@labenu-bot labenu-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
Há duas áreas separadas no site, uma destinada à criação de anúncios de produtos e outra destinada à busca desses anúncios
Há um carrinho que o usuário possa adicionar serviços
O usuário é capaz de visualizar uma lista de serviços
O usuário é capaz de visualizar os dados do serviço (pelo menos nome e preço)
O usuário é capaz de filtrar os serviços por valor mínimo e máximo
O usuário é capaz de buscar os serviços por título ou descrição
O usuário é capaz de ordenar os serviços por nome
O usuário é capaz de ordenar os serviços por preço
O usuário é capaz de ordenar os serviços por pelo prazo
O usuário deve ser capaz de adicionar filtros, busca e ordenação simultaneamente -
O usuário deve ser capaz de retornar à visualização sem filtros, mostrando todos os serviços disponíveis
O usuário é capaz de adicionar um serviço no carrinho
O usuário é capaz de remover um serviço do carrinho
O usuário é capaz de ver corretamente o valor total de sua compra
O usuário é capaz de inserir serviços por um formulário com os seguintes campos: Nome, Descrição, Preço, Método(s) de pagamento aceito(s), Prazo.
É possível escolher mais de um método de pagamento simultaneamente'
Os campos devem ser limpos assim que o serviço for adicionado
Um aviso na tela deve indicar se foi possível adicionar o serviço ou não
Os serviços adicionados pelo formulário devem aparecer na lista de serviços disponíveis
Quão fácil foi achar as informações no site? Médio
O que você achou do design do site? Bom
[Desafio] Responsividade para celulares
[Desafio] Colocar Loadings sempre que estiver carregando alguma requisição -
[Desafio] Manter os itens no carrinho mesmo que a página seja atualizada
[Desafio] Não permitir que um mesmo serviço seja adicionado ao carrinho mais de uma vez
Funcionalidade extra -
Feedback do código Feito
Boa indentação
Boa nomenclatura de variáveis e funções, utilizando camelCase
Não deixou nenhum log ou registro desnecessário pelo código
Isolou constantes que se repetem em arquivos separados -
Erros são notificados ao usuário de forma clara para o usuário, e não apenas no console
Estilização bem organizada (separada em arquivos para cada componente/página e não junto com os componentes) -
Requisições declaradas fora das funções de ciclo de vida (componentDidMount, componentDidUpdate)
Criou link do surge
Criou um bom readme com informações relevantes para o projeto

Análise realizada por: Janaylla

@Janaylla Janaylla changed the title PR de Corrção PR de Correção Oct 4, 2021
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

Successfully merging this pull request may close these issues.

7 participants