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

Feat/feature criar layout module #55 #63

Merged
merged 4 commits into from
Jun 20, 2024

Conversation

tbatista86
Copy link
Contributor

Descrição

--- Criado modulo de layout onde serão implementados componentes do portal, tais como header, footer entre outros.
para criação do mesmo foi utlizado o NX com comando [npx nx generate @portal-ascepa/angular-plugin:module --name=layout]

Tipo da alteração

feat: Uma nova funcionalidade

--- FEAT criando modulo layout

Checklist

  • Revi meu próprio código antes de enviar para revisão
  • Testei minha correção ou recurso extensivamente antes de enviar para revisão
  • Formatei meu código usando o guia de estilo de interface (Prettier + ESLint)
  • Meu código está bem documentado com apenas informações relevantes
  • Minhas alterações não geram novos avisos
  • Meus commits são bem organizados, descritivos e fáceis de reverter
  • Adicionei testes que provam que minha correção é eficaz ou que meu recurso funciona
  • Meu código está seguindo os padrões de acessibilidade

Outras observações

--- adiciona commentário ao README do modulo cirado

Copy link

github-actions bot commented Jun 15, 2024

Deploy preview for portal-ascepa ready!

✅ Preview
https://portal-ascepa-f12ymzv4r-andrew-rosarios-projects.vercel.app

Built with commit 52a7a55.
This pull request is being automatically deployed with vercel-action

Copy link
Contributor

@andrewarosario andrewarosario left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

modules/layout/src/lib/layout/layout.component.ts Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
modules/layout/README.md Outdated Show resolved Hide resolved
@andrewarosario andrewarosario linked an issue Jun 17, 2024 that may be closed by this pull request
2 tasks
@tbatista86 tbatista86 added the refactor Changes that do not affect external behavior label Jun 17, 2024
@tbatista86 tbatista86 modified the milestone: Layout Shared Jun 17, 2024
Copy link
Member

@Werisu Werisu left a comment

Choose a reason for hiding this comment

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

Com base na revisão e nos testes realizados, aprovo a integração deste módulo ao branch principal.

Parabéns pelo excelente trabalho!

Copy link

@willflame willflame left a comment

Choose a reason for hiding this comment

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

mandei uma duvida para possível padronização para documentação

Choose a reason for hiding this comment

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

seria bom verificar para não deixar talvez redundante esta parte da documentação:

layout

ou

Module Layout

Joguei a duvida no Discord

@tbatista86 tbatista86 merged commit eb88c26 into main Jun 20, 2024
3 checks passed
@tbatista86 tbatista86 deleted the feat/feature-criar-layout-module-#55 branch June 20, 2024 15:26
@tbatista86
Copy link
Contributor Author

m[Merge PR to create module layout in project to create modules like header and footer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Changes that do not affect external behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Criar layout module
4 participants