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

26/pagina detalhe tema #38

Merged
merged 9 commits into from
Aug 8, 2024
Merged

26/pagina detalhe tema #38

merged 9 commits into from
Aug 8, 2024

Conversation

Wander06
Copy link
Contributor

@Wander06 Wander06 commented Aug 6, 2024

#26 - Página detalhe tema

🆙 CHANGELOG

  • Adiciona detalhamento do tema
  • Layout responsivo
  • Espaçamento entre cards
  • Estrutura de grid para cards
  • Responsividade para textos
  • Adiciona container pai, onde não permite expandir a página a partir de um certo tamanho

⚠️ Me certifico que:

  • Não deixei nenhum novo warning, erro ou console.log nas minhas modificações
  • Fiz deploy para ambiente de teste certificando que o build não quebrou
  • Solicitei code review para 2 pessoas
  • Solicitei QA para 2 pessoas
  • Obtive aprovação de QA e posso fazer merge

⚠️ Como testar:

  • Dar gitpull na main e na branch 26
  • Executar 'npm run dev'
  • Acessar algum tema
  • Verificar se os componentes aparecem corretamente "Titulo, descrição, e cards de tópicos"
  • Verificar responsividade da pagina
  • Verificar se a NavBar não se sobrepoe aos elementos
  • Aplicação não deve conter nenhum erro, warning ou console.log
  • Alteração proposta no card foi implementada


function TopicsSeparator(topics: string) {
const topic = topics.split(",");
return topic;
Copy link
Member

Choose a reason for hiding this comment

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

Por enquanto deixamos assim, mas depois será que não conseguimos aplicar mais uns conceitos de responsabilidade e refatorar?

@barbaraanger barbaraanger enabled auto-merge (rebase) August 8, 2024 12:45
auto-merge was automatically disabled August 8, 2024 12:48

Rebase failed

@barbaraanger barbaraanger reopened this Aug 8, 2024
@barbaraanger
Copy link
Member

(só fechei e abri pra testar a workflow)

@barbaraanger barbaraanger linked an issue Aug 8, 2024 that may be closed by this pull request
4 tasks
@barbaraanger barbaraanger merged commit ec55c71 into main Aug 8, 2024
1 check passed
@barbaraanger barbaraanger deleted the 26/pagina-detalhe-tema branch August 8, 2024 13:33
@Wander06 Wander06 mentioned this pull request Sep 2, 2024
10 tasks
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.

Página: Detalhamento do Tema
4 participants