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

Implementação do alerta de resub #10

Merged

Conversation

Limatucano
Copy link
Contributor

@Limatucano Limatucano commented Jan 14, 2022

Implementação da funcionalidade de escutar o evento de resub no canal.

Eu acredito que essa não seja a melhor forma de ser feita, podemos agrupar os eventos em um object e acessar pela propriedade, já que o comportamento a principio é o mesmo ou colocar dentro de uma function recebendo alguns params como valor de opacidade e time.

Mas o que você acha?

Closes #1

Copy link
Owner

@jlcarvalho jlcarvalho left a comment

Choose a reason for hiding this comment

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

Primeiramente eu gostaria de agradecer por você ter submetido o PR, fiquei realmente surpreso e feliz quando vi ele aqui. Obrigado 😄

Eu fiz algumas sugestões no código, algumas relacionadas a bugs e outras relacionadas a melhorias de legibilidade mesmo, assim que você ajustar eu faço o merge aqui.

assets/alerts.html Outdated Show resolved Hide resolved
assets/alerts.html Outdated Show resolved Hide resolved
assets/alerts.html Outdated Show resolved Hide resolved
assets/alerts.html Outdated Show resolved Hide resolved
assets/alerts.html Show resolved Hide resolved
@Limatucano
Copy link
Contributor Author

cara, que legal. Eu nunca tive essa interação de code review, sempre trabalhei sozinho kkkk então estou bem animado.
Vou dar uma olhada nos pontos que vc comentou, ai preciso abrir um novo PR? Ou tem como atrelar um commit aqui?

@Limatucano
Copy link
Contributor Author

acho que consegui fazer o commit, se nao conseguir ver me da um alo.
Sobre as mudanças que sugeriu achei ótimas, na função animateBox eu adicionei um param para duraçao do box, caso a gente precise que algum seja mais rapido ou algo do tipo, mas o default está 8000

@jlcarvalho
Copy link
Owner

@Limatucano show de bola cara, amanhã na live eu faço o review novamente, pode ser?

@jlcarvalho jlcarvalho changed the title Feat/implementacao alerta resub #1 Implementação do alerta de resub Jan 20, 2022
@Limatucano
Copy link
Contributor Author

beleza @jlcarvalho. Vou ficar ligado no twitter para saber da live, mas tem ideia de que horas vai começar?

@jlcarvalho
Copy link
Owner

@Limatucano toda terça, quinta e sábado às 19h.

Copy link
Owner

@jlcarvalho jlcarvalho left a comment

Choose a reason for hiding this comment

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

image

@jlcarvalho jlcarvalho merged commit e77c632 into jlcarvalho:main Jan 20, 2022
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.

[FEAT] Implementar alerta de resub
2 participants