Skip to content

Conversation

@JoaoPauloJorgeDeOliveira

Após fazer o curso na Udemy, percebi que o código no GitHub não era o mais atualizado.
Assim, atualizei com os 2 métodos faltantes.
Além disso, criei 3 novos métodos, visando:

  1. Obter a lista com os nomes completos dos contato dentro de um grupo (get_group_members_long)
  2. Deletar uma mensagem recente (delete_message_from_recent).
  3. Pegar todas as mensagens visíveis (os elementos, não apenas os textos), visando conseguir o método em (2) (get_all_messages_elements).

@WittmannF
Copy link
Owner

Fala @JoaoPauloJorgeDeOliveira ! Eu não esqueci de suas excelentes contribuições! Só estou pensando em como fazer elas ao mesmo tempo que mantenho o arquivo original do curso intacto para que não confundam os alunos. Uma ideia que tive aqui seria passar essas sugestões de melhoramentos para um segundo arquivo, talvez poderíamos chamar ele de whatsapp_api_advanced.py ou similar. Aí eu posso acrescentar um comentário no curso sobre essas atualizações. O que vc acha?

@JoaoPauloJorgeDeOliveira
Copy link
Author

Boa noite, @WittmannF !
Pessoalmente, eu não acho que ficaria confuso manter apenas 1 arquivo.
Meu primeiro commit era apenas acrescentando métodos que você já explica no curso. Eles só não estavam aqui no código.
Já o segundo realmente implementa 3 métodos novos. Porém, acho que o aluno que estiver fazendo o curso simplesmente se concentraria nos métodos que você explica nas aulas e ignoraria os novos.
Se eu estivesse fazendo o curso neste momento, acho que eu ficaria com mais dúvidas se houvessem 2 arquivos (e eu não soubesse qual deles usar) do que se houvesse um único arquivo com uma parte não abordada nas aulas.
Além disso, acho que há a opção de colocar um comentário escrito (ou um vídeo de 2 minutos) no curso apenas avisando que o código foi atualizado.
Porém, fique a vontade para proceder como achar melhor. 👍

@WittmannF
Copy link
Owner

WittmannF commented May 8, 2020 via email

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.

2 participants