search

comune-desio / azuredesio

GNU General Public License v3.0
0 stars 0 forks source link

Funzionalità di recupero dati da URL #3

Open olistik opened 8 years ago

olistik commented 8 years ago
olistik commented 8 years ago

@ipersoft ho impostato TextFromURL a https://info.desio.org per il menù HOME. Quando faccio partire il bot non vedo alcuna modifica, poi accedo al sottomenù "Menù mensa scolastica" e seleziono il bottone "Menu precedente". Lo spinner funziona per un po' e poi si toglie, senza mostrarmi altro contenuto. Dove pensi possa essere il problema?

ipersoft commented 8 years ago

Per i test mettiamo un tasto Menu con il sottomenu Situazione Corrente?

Perchè vorrei aggiungere anche previsioni e allerte in corso.

@olistik @favoeva

olistik commented 8 years ago

@ipersoft dovremmo prima verificare che l'integrazione delle modifiche funzioni.

ipersoft commented 8 years ago

@olistik quindi come si dovrebbe procedere?

olistik commented 8 years ago

@ipersoft

  1. verificando la presenza del bug che ho menzionato qua sopra
  2. definendo il risultato atteso della presenza di TextFromURL in un contesto (Context)
  3. inserendo un TextFromURL in un contesto e verificando che il risultato atteso (2.) corrisponda al risultato ottenuto.
ipersoft commented 8 years ago

mettendo a mano TextFromURL sul bot demo https://github.com/ipersoft/telegram_bot/blob/master/botdemo.json

http://telegram.me/DemoIpersoftBot

Funziona

olistik commented 8 years ago

Ho provato un "happy path": menu "Dev test" accessibile da HOME il cui contesto contiene TextFromURL = http://desio.azurewebsites.net/plugindesio/now.php e funziona:

screen shot 2016-09-16 at 11 43 58

Ho quindi identificato meglio il caso che genera il bug che ti ho descritto prima. In realtà i bug sono due, dammene conferma:

  1. TextFromURL a http://www.google.com non funziona
  2. quando l'utente Telegram fa partire il bot con il comando di start, il codice non effettui alcun controllo sulla presenza del campo TextFromURL e si limita quindi a mostrare il contenuto di Text

Una volta risolti questi due problemi possiamo chiudere la issue.

olistik commented 8 years ago

@ipersoft intanto sposto questo in una issue separata:

Per i test mettiamo un tasto Menu con il sottomenu Situazione Corrente? Perchè vorrei aggiungere anche previsioni e allerte in corso.

olistik commented 8 years ago

Nel frattempo tolgo il menù "Dev test".

ipersoft commented 8 years ago

@olistik

  1. Nel caso uso vorresti il codice tutto il codice della pagina html di www.google.com?
  2. Stai parlando del TextFromURL nel menu HOME?
olistik commented 8 years ago

@ipersoft

  1. Nel caso uso vorresti il codice tutto il codice della pagina html di www.google.com?

Sì, sarebbe il risultato che mi aspetterei da una TextFromURL. Ma se il comportamento è un altro allora basta definirlo e comportarsi di conseguenza :-) Non sapendo come funziona non posso neanche comprendere quali sono i motivi dello stallo verificatosi nel mio test.

  1. Stai parlando del TextFromURL nel menu HOME?

Yep.

ipersoft commented 8 years ago

Il Text From URL mi sembra di averlo attivato in call e non nel messaggio iniziale. Per questo non si attiva subito ma solo in call. Per il fatto che non ti ha restituito niente e perché passando il testo a telegram deve essere andato in errore o per i tag o per il limite del testo.