Skip to content
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

Feature - Marketplace Pages #535

Merged
merged 34 commits into from
Sep 14, 2021
Merged

Feature - Marketplace Pages #535

merged 34 commits into from
Sep 14, 2021

Conversation

LuanEdCosta
Copy link
Contributor

@LuanEdCosta LuanEdCosta commented Sep 3, 2021

  • Create the marketplace initial page
  • Create the marketplace task details page
  • Create the marketplace search/tasks page
  • Create the marketplace author page
  • Create the marketplace API
  • Create the marketplace store
  • Make adjustments in routes and other components to integrate with the new pages

All pages are responsive, but we need to create a Header component and make it responsive too

@github-actions
Copy link

github-actions bot commented Sep 3, 2021

Dicas para revisão de código

Commits

  • Título (1a linha do commit): apresentar resumo do que foi alterado/adicionado/removido.
    ex: adiciona action que salva parametros no backend; exibe rótulo no componente de selecao de dataset;
  • Descrição (outras linhas): dar mais detalhes de cada alteração:
    • motivos das alterações
      ex: havia um bug que causava...; nova funcionalidade que faz isso...; código foi movido para...;
    • bibliotecas adicionadas e versões (package.json)
      ex: atualiza para antd v4.6;
    • testes unitários criados/alterados
      ex: adiciona testes para a action fetchProjectSuccess;
  • Mensagens auto-explicativas! Quem revisa o código deve entender o que foi feito (e porque foi feito) sem perguntar para quem fez o commit.
  • Não devem ter conflitos. Solicitar que sejam resolvidas as ocorrências de "This branch has conflicts that must be resolved".

SonarCloud Quality Gate

  • Coverage > 80.0%, e sempre que possível = 100%
  • 0 Bugs, 0 Code Smells, 0 Vulnerabilities

Build Github actions COM SUCESSO

ReactJS

  • Usar Node 12 ou 14.
  • Remover console.log.
  • Não deixar código-fonte comentado.
  • Pasta de módulos devem ser escritas em lower case
  • Pasta de componentes devem ser escritas em camel case
  • Nunca utilizar spread em retorno de reducer sem alteração
  • Sempre utilizar hooks ao invés de HOCs
  • Itens em uma lista devem ser renderizados com uma chave única (preferir uuid)
  • Comentar apenas códigos que não estão muito claros
  • Não utilizar comentários para blocos
  • Action types devem conter @

Redux

As instruções a seguir foram retiradas do Redux Style Guide.

@LuanEdCosta LuanEdCosta changed the title Feature - Marketplace Page Initial Page Feature - Marketplace Initial Page Sep 3, 2021
@LuanEdCosta LuanEdCosta changed the title Feature - Marketplace Initial Page Feature - Marketplace Pages Sep 8, 2021
@LuanEdCosta LuanEdCosta marked this pull request as draft September 8, 2021 12:11
@LuanEdCosta LuanEdCosta marked this pull request as ready for review September 14, 2021 01:24
Copy link
Member

@fberanizo fberanizo left a comment

Choose a reason for hiding this comment

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

👍 @LuanEdCosta Tudo okay!
Provavelmente vamos fazer ajustes nas URL (acho que o marketplace deve usar um prefixo diferente de projects/apis/marketplace), mas isso faremos depois que o backend estiver pronto.

Só estou em dúvida se mantemos o item de menu visível, ou escondido por enquanto...

@LuanEdCosta
Copy link
Contributor Author

Ah certamente alguns ajustes terão que ser feitos quando for feita a integração com a API 😅.
Sobre o item do menu vou esconder ele por aqui 👍, já subo as alterações neste mesmo PR

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

16.0% 16.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@fberanizo fberanizo left a comment

Choose a reason for hiding this comment

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

Testado com sucesso no amb .53
image
image

@fberanizo fberanizo merged commit a79bcf5 into master Sep 14, 2021
@LuanEdCosta LuanEdCosta deleted the feature/marketplace-page branch February 22, 2022 13:15
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.

2 participants