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.

Sign up for GitHub

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

Jump to bottom

Aggiungere un link al messaggio telegram al commento del bot quando posta un testo #22

Closed

zarEclEC opened this issue Jan 29, 2019 · 6 comments

Assignees

Labels

Delayed enhancement

Milestone

Contributor

zarEclEC commented Jan 29, 2019

In riferimento a quanto ho scritto in #20, prendendo questo commento di test
https://www.reddit.com/r/ItalyInformatica/comments/al2edh/telegram_delooo_provissima/efa0lbb/

Sarebbe bello cambiare il link delle parole "gruppo telegram" da https://t.me/ItalyInformatica a https://t.me/ItalyInformatica/1234 dove 1234 è il message_id del messaggio da cui proviene il post

ScrappyCocco added the enhancement label

ScrappyCocco added this to the Versione 3.0 milestone

ScrappyCocco added the Delayed label

ScrappyCocco assigned zarEclEC

Contributor Author

zarEclEC commented Jan 30, 2019

Domani dopo il mio esame (che ho alla mattina), nel pomeriggio, che ho un po' di tempo, mi ci metto 👍

Contributor Author

zarEclEC commented Jan 31, 2019

Forse fatto qui https://github.com/ArmeF97/marvin/commit/cd5189972879100dca1259bd21105f41761d8fdd
@ScrappyCocco

Contributor Author

zarEclEC commented Jan 31, 2019 •

edited

Loading

Lascia perdere che per sbaglio nel commit ho cancellato l'example ahaha (che fa il suo ritorno qui https://github.com/ArmeF97/marvin/commit/ca1dd47ce64e791c1493f56b4783a72e4a268a3f )

Collaborator

ScrappyCocco commented Jan 31, 2019 •

edited

Loading

No così con due file è illeggibile per carità
Piuttosto metti un link al gruppo telegram nel json e nel file di testo metti una stringa tipo {link}, così poi leggi dal json, attacchi l'id e poi sostituisci nel testo

Contributor Author

zarEclEC commented Jan 31, 2019

Ok, faccio

Contributor Author

zarEclEC commented Jan 31, 2019

Direi che con l'ultimo commit dovremmo esserci
https://github.com/ArmeF97/marvin/commit/a8cb6abf261bcb59c6b94c15089df29101ebd290
@ScrappyCocco

zarEclEC mentioned this issue

Merged

ScrappyCocco added a commit that referenced this issue


          Merge pull request #29 from ArmeF97/version_3

ae681b2

#22 e #23

ScrappyCocco closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment