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: date input component #169

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

SamyrOR
Copy link
Contributor

@SamyrOR SamyrOR commented Jun 11, 2024

Descrição

Descrição do que é esperado deste PR
Input de seleção de data

Stories relacionadas (Shortcut)

  • [sc-xxxx]

Pontos para atenção

  • Listar pontos para atenção no review
  • Listar pontos para atenção nos testes

Possui novas configurações?

  • Descrever as configurações alteradas ou novas

Possui migrations?

  • Se a feature adicionou alguma migration e como faz para rodar

@SamyrOR
Copy link
Contributor Author

SamyrOR commented Jun 12, 2024

Olá @zoedsoupe!

Não consegui entrar em contato com você pelo telegram, nem pelo linkedin kk
Participei do elixir days, e na sua palestra (alias, parabéns!) você tinha comentado desse projeto e das issues
Sou dev front-end e como to aprendendo elixir, resolvi contribuir criando esse input de data que ta la nas issues

Qualquer coisa só dar um toque =)

@zoedsoupe
Copy link
Member

olá samyr! primeiramente e desde já obrigada pela contribuição para com o projeto, o intuito é de fato incentivar a comunidade elixir, com foco em quem está migrando de stack ou está iniciando os estudos em programação web com elixir.

peço desculpas pela demora na resposta, acabei tendo alguns imprevistos no trabalho e tenho tido bem menos tempo pra dedicar ao projeto, mas saiba que sua contribuição tem grande importância!

Copy link
Member

@zoedsoupe zoedsoupe left a comment

Choose a reason for hiding this comment

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

gostei bastante da implementação, seguiu o padrão do projeto, mesmo introduzindo mais uma dependência via frontend (o que não tem problema nenhum). implementação sucinta e que bate com o layout no figma.

única coisa que eu pediria para caso faça próximas contribuições é incluir um screenshot, caso seja um tela ou um componente, do resultado final.

@zoedsoupe zoedsoupe merged commit 474548c into peapescarte:main Jul 11, 2024
@SamyrOR
Copy link
Contributor Author

SamyrOR commented Jul 16, 2024

Sem problemas @zoedsoupe!
Gostei bastante em poder contribuir
Acabei incluindo a dependência no front, por que quando fiz o input do 0 senti falta de algumas features, que pacotes como esse ja traziam.
Obrigado pelo feedback, sobre o screenshot, pode deixar que nas próximas vai constar =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Completo
Development

Successfully merging this pull request may close these issues.

2 participants