Python API: chatbot ziet nieuwe berichten in de app niet

[kaypee]

Mijn vraag
Ik probeer een chatbot te schrijven, die afhankelijk van de meest recente post in een Webex chatroom een bepaalde reply post.

Dat werkt, script blijft ook lopen in een loop.

Maar als ik de Webex app open doe en in die chatroom zelf een nieuw bericht post, wordt dat niet opgepikt door het script.


Relevante software en hardware die ik gebruik
Webex API, TheDogAPI

Wat ik al gevonden of geprobeerd heb:
Volgens mij herstart de loop wel maar worden de messages niet opnieuw ingeladen via de laatste requests.get, waar ik nochtans om vraag.

Het probleemgedeelte begint vanaf line 112 hieronder:

code:

Code weggehaald

[Voor 83% gewijzigd door kaypee op 10-11-2022 19:37. Reden: Code weggehaald]

[Chaseream]

Het haalt niet jou berichten op omdat je de berichten api van webex 1x aanroept op lijn 85 en dat is buiten de loop. Wat er dus gebeurt is dat het de berichten 1 keer ophaalt als je de script start.

Op lijn 118 zet je de opgehaalde response van de webex api alleen om in een json object.

Wat je kan doen is een nieuwe functie maken die de webex api call doet en de messages returned en dat elke keer in de while loop doen.

code:

def getMessagesFromWebex:
    r_messages = requests.get(   messagesAPI,
       ...             
    if not r_messages.status_code == 200:
        ...
    # get the JSON formatted returned data
    json_data = r_messages.json()
    
    # check if there are any messages in the "items" array
    if len(json_data["items"]) == 0:
        ...
    
    
    # store the array of messages
    return json_data["items"]
    
while True:
    ....
    messages = getMessagesFromWebex()

ps ik heb al een tijdje geen python gedaan

[Robicide]

Ik denk dat je het blok vanaf regel 168 waarin je nieuwe messages ophaalt minder moet indenten, nu is het onderdeel van je elif waardoor het alleen wordt uitgevoerd als er al nieuwe berichten zijn.

[kaypee]

Robicide schreef op donderdag 10 november 2022 @ 18:45:
Ik denk dat je het blok vanaf regel 168 waarin je nieuwe messages ophaalt minder moet indenten, nu is het onderdeel van je elif waardoor het alleen wordt uitgevoerd als er al nieuwe berichten zijn.

Thanks a lot!
Getest met de indent 1 tab in te korten.. works.

Ook bedankt @Chaseream , jouw oplossing werkte ook.

[mbe81]

@kaypee Jammer dat je de code in de OP weg hebt gehaald. Nu kan niemand meer zien wat er/ waarom het in de oorspronkelijke code mis ging en is het topic voor andere tweakers waardeloos geworden

[.oisyn]

Ja, dat lijkt me niet de bedoeling, @kaypee. Dit is geen persoonlijke afhaalbalie, we maken dit forum met z'n allen en dus is het ook van belang dat iedereen ervan kan leren. Hierbij het verzoek om je code in het vervolg gewoon in het topic te laten staan.

Ik gooi 'm nu op slot omdat hij compleet nutteloos is geworden.