Skip to content

Bugfix/slot on cds alert#1079

Merged
uesley merged 4 commits intomainfrom
bugfix/slot-on-cds-alert
Apr 15, 2026
Merged

Bugfix/slot on cds alert#1079
uesley merged 4 commits intomainfrom
bugfix/slot-on-cds-alert

Conversation

@IhGori
Copy link
Copy Markdown
Contributor

@IhGori IhGori commented Apr 15, 2026

Por favor, verifique se o seu pull request está de acordo com o checklist abaixo:

  • A implementação feita possui testes (Caso haja um motivo para não haver testes/haver apenas testes de snapshot, descrever abaixo)
  • A documentação no mdx foi feita ou atualizada, caso necessário
  • O eslint passou localmente

1 - Resumo

  • Corrige o não funcionamento do slot para este componente;
  • Atualiza md em que caso de uso indicava uma variação que não existe para esse componente;
  • Adiciona validador do tipo de variação para prop variant

2 - Tipo de pull request

  • 🧱 Novo componente
  • ✨ Nova feature ou melhoria
  • 🐛 Fix
  • 👨‍💻 Refatoração
  • 📝 Documentação
  • 🎨 Estilo
  • 🤖 Build ou CI/CD

3 - Esse PR fecha alguma issue? Favor referenciá-la

Não

4 - Quais são os passos para avaliar o pull request?

  • Acesse o componente verifique que continua a funcionar a declaração da prop text;
  • Altere o alert.md para testar o funcionamento do slot:
<PreviewBuilder
    :args="args"
    :component="CdsAlert"
>
         Oi, estou usando slot!
</PreviewBuilder>
  • Verifique que foi exibido corretamente o conteúdo interno com slot.

5 - Imagem ou exemplo de uso:

image

6 - Esse pull request adiciona breaking changes?

  • Sim
  • Não

@github-actions github-actions Bot added the 🐛 Bug Algo não está funcionando label Apr 15, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 15, 2026

Greptile Summary

Este PR corrige o funcionamento do slot no componente CdsAlert, substituindo o padrão v-if/v-else por um slot com {{ text }} como conteúdo fallback — abordagem idiomática do Vue. Também adiciona um validador para a prop variant e corrige o exemplo de documentação que usava variant="green" (inexistente).

  • A variante success ainda possui tratamento no watch e classes CSS no template (ex.: alert__container--success, alert__icon--success), mas agora é rejeitada pelo novo validador. Defina explicitamente se success é suportada ou não, e remova o código morto correspondente.

Confidence Score: 5/5

PR seguro para merge — a correção do slot é correta e os demais ajustes são consistentes.

A única ressalva é a inconsistência entre o validador e o código de suporte à variante success, que é uma questão de P2 (código morto/inconsistência de documentação) e não bloqueia o merge.

src/components/Alert.vue — verificar se a variante success deve ser mantida ou removida do watch e do template.

Important Files Changed

Filename Overview
src/components/Alert.vue Corrige o slot usando o padrão de fallback do Vue (slot com {{ text }} como conteúdo padrão); adiciona validador para variant; variante success permanece no template/watch mas é rejeitada pelo validador.
docs/.docgen/components-metadata.json Atualiza metadado do prop text: remove required: false (redundante) e ajusta o valor padrão de 'Título do AlertCard' para '', espelhando a mudança no componente.
docs/components/notificação/alert.md Corrige exemplo de uso substituindo variant="green" (inválida) por variant="info".

Reviews (1): Last reviewed commit: "refactor: Corrige exemplo de uso do com..." | Re-trigger Greptile

Comment thread src/components/Alert.vue Outdated
@uesley uesley merged commit 8382ecd into main Apr 15, 2026
8 checks passed
@uesley uesley deleted the bugfix/slot-on-cds-alert branch April 15, 2026 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Bug Algo não está funcionando

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants