14 socials integration #34

2025-10-19T12:21:56+02:00

Nunzi99 commented

2025-10-19 12:21:56 +02:00

(Migrated from github.com)

No description provided.

Berack96 (Migrated from github.com) requested changes 2025-10-19 16:22:36 +02:00

.env.example

Berack96 (Migrated from github.com) commented

2025-10-19 16:11:21 +02:00

Vanno bene le chiavi di provider diversi, ma non è da fare in questa PR.
Fai un branch nuovo in cui ci sono queste modifiche

Vanno bene le chiavi di provider diversi, ma non è da fare in questa PR. Fai un branch nuovo in cui ci sono queste modifiche

pyproject.toml

Berack96 (Migrated from github.com) commented

2025-10-19 16:12:15 +02:00

Come per il file x.py, è meglio usare direttamente il comando della CMD

src/app/api/social/chan.py

Berack96 (Migrated from github.com) commented

2025-10-19 16:18:28 +02:00

Pulire il file che ha troppi commenti, in modo da rendere il codice più leggibile
Rimuovere inoltre la quantità di try non necessaria, dato che se l'API ha dei problemi non voglio che mi restituisca un oggetto vuoto, ma voglio l'errore

Pulire il file che ha troppi commenti, in modo da rendere il codice più leggibile Rimuovere inoltre la quantità di try non necessaria, dato che se l'API ha dei problemi non voglio che mi restituisca un oggetto vuoto, ma voglio l'errore

src/app/api/social/chan.py

						
				@@ -0,0 +1,89 @@

				'''

Berack96 (Migrated from github.com) commented

2025-10-19 16:20:21 +02:00

per evitare il try usa thread.get('sub', '') così non da errore e restituisce solo vuoto se non cè

per evitare il try usa ```thread.get('sub', '')``` così non da errore e restituisce solo vuoto se non cè

src/app/api/social/x.py

						
				@@ -0,0 +1,46 @@

				import os

Berack96 (Migrated from github.com) commented

2025-10-19 16:17:02 +02:00

Direi che è meglio non usare docker direttamente per questa implementazione...
Preferirei che controllassi se il comando rettiwt è presente nel sistema, e se non lo è dare un errore.

Qualcosa tipo questo, formattato meglio e controlato se funziona:

from shutil import which
assert which('rettiwt') is not None, "Command `rettiwt` not installed"

Direi che è meglio non usare docker direttamente per questa implementazione... Preferirei che controllassi se il comando `rettiwt` è presente nel sistema, e se non lo è dare un errore. Qualcosa tipo questo, formattato meglio e controlato se funziona: ```python from shutil import which assert which('rettiwt') is not None, "Command `rettiwt` not installed" ```

src/app/models.py

Berack96 (Migrated from github.com) commented

2025-10-19 16:10:14 +02:00

Modifica non necessaria, dato che il main ha già fatto degli aggiornamenti su questa sezione e c'è già una altra Issue marcata per queste cose

Berack96 (Migrated from github.com) approved these changes 2025-10-20 16:54:32 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Berack/upo-app-agents#34

				`@@ -0,0 +1,89 @@`
				`'''`

				`@@ -0,0 +1,46 @@`
				`import os`