Log channel evolution#21
Conversation
| if log_chat: | ||
| result = f'<b>Risorsa inviata da @{update.effective_user.username}:</b>\n' | ||
| for descriptor, entity in entities.items(): | ||
| if descriptor['type'] == MessageEntity.HASHTAG: |
There was a problem hiding this comment.
Questo controllo è possibile farlo nel for loop della riga 62 ? visto che è lo stesso identico ciclo, almeno risparmiamo un'iterazione
| response = requests.get(entity) | ||
| if response.status_code == requests.codes.ok: | ||
| result += f'{entity}' | ||
| extracted = tldextract.extract(entity) |
There was a problem hiding this comment.
Questa parte di estrazione del dominio e costruzione dei tags viene ripetuta più volte per le varie casistiche di MessageEntity, che ne pensi di provare a generalizzare quel flusso in qualche funzione di util così eviteresti un po' di codice ripetuto.
| response = requests.get(descriptor['url']) | ||
| if response.status_code == requests.codes.ok: | ||
| result += f'{descriptor["url"]}' | ||
| extracted = tldextract.extract(descriptor['url']) |
There was a problem hiding this comment.
Tipo qua si ripete lo stesso flusso ( aggiungo questo commento così si capisce meglio cosa intendo 😆
|
Questa va ribasata su master visto che abbiamo mergiato la mia |
iflare3g
force-pushed
the
log_channel_evolution
branch
from
8834acd to
5159c39
Compare
|
L'ho ribasata così ora la history è in linea e infatti i conflitti sono risolti 🚀 |
|
Ciao a tutti, bisogna integrare nel file requirements.txt il seguente modulo:
|
3 participants
Richiesta di merge per gli sviluppi sul modulo log channel, ovvero l'implementazione del log di risorse URL con annessa verifica della validità del link e possibilità di allegare alla risorsa degli hashtag con l'obiettivo di rendere la risorsa tracciabile e dare indicazioni descrittive minimali. Viene inoltre generato un hashtag a partire dal nome del sito web presente nella URL. Nel caso la risorsa sia un TEXT_LINK, allora viene recuperata la URL mascherata e utilizzata come risorsa.