Skip to content

Conversation

@MattiaFontana997
Copy link

🔧 Fix: Problema caricamento cartella config, notifiche Alexa e errori regex

📋 Descrizione

Questa PR risolve tre problemi principali:

  • Il caricamento errato della cartella config nei moduli del notifier.
  • La mancata consegna delle notifiche vocali e push via Alexa.
  • Alcuni errori di sintassi nelle espressioni regolari (regex) usate nel codice.

✅ Modifiche principali

  • Refactor della variabile my_config_dir per usare Path in modo corretto e portabile.
  • Corretto l'utilizzo di call_service() per Alexa con service_data strutturato correttamente.
  • Sistemati alcuni syntax error nelle regex per evitare crash o comportamenti inattesi.
  • Gestione migliorata della configurazione dei volumi automatici per i dispositivi Alexa.

🧪 Come testare

  1. Assicurati che il modulo notifier sia correttamente configurato.
  2. Invia una notifica vocale e push ad un dispositivo Alexa.
  3. Verifica che:
    • la notifica venga ricevuta correttamente.
    • la configurazione venga caricata senza errori dal path giusto.
  4. Verifica che non ci siano più errori legati a espressioni regolari nel log.

📝 Note

  • Testato con Home Assistant in ambiente Proxmox + AppDaemon.
  • Nessuna breaking change introdotta, compatibile con configurazioni esistenti.

@caiosweet
Copy link
Contributor

Ma è stato fatto con chatGPT?

@MattiaFontana997
Copy link
Author

Ma è stato fatto con chatGPT?

Cosa?

@caiosweet
Copy link
Contributor

Ma è stato fatto con chatGPT?

Cosa?

Le fix. questa pullrequest...

@MattiaFontana997
Copy link
Author

Ma è stato fatto con chatGPT?

Cosa?

Le fix. questa pullrequest...

No 😳 perché? 😅😅
Per quel errore nella regex dici?

@caiosweet
Copy link
Contributor

Ma è stato fatto con chatGPT?

Cosa?

Le fix. questa pullrequest...

No 😳 perché? 😅😅 Per quel errore nella regex dici?

Esatto, tipo (r"&", " rand "), # escape e altre mancano r
Comunque la sto confrontando con quella che avevo fatto io, grazie per l'aiuto :0)

@MattiaFontana997
Copy link
Author

Ma è stato fatto con chatGPT?

Cosa?

Le fix. questa pullrequest...

No 😳 perché? 😅😅 Per quel errore nella regex dici?

Esatto, tipo (r"&", " rand "), # escape e altre mancano r

Comunque la sto confrontando con quella che avevo fatto io, grazie per l'aiuto :0)

Vista proprio adesso mentre ti scrivevo qui 😂 stavo infatti vedendo nei commit precedenti da dove mi era uscita rand 😞

@caiosweet
Copy link
Contributor

Se può interessare, vedevo qualche feedback, nell'altro progetto, dove a quanto pare va tolto anche return_result=True dal file py di alexa...

@MattiaFontana997
Copy link
Author

Se può interessare, vedevo qualche feedback, nell'altro progetto, dove a quanto pare va tolto anche return_result=True dal file py di alexa...

Si eliminandole sparisce l'errore nel log , anche perchè sono superflue essendo che quelle due function sono void.
Fatta qualche prova con Alexa,Persistenti e Testo funziano tutte. Da testare la parte VOIP e Google

Fixed Last Error of Now Allowed Key
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