feat: implement database to store users email #8
Conversation
fix: use absolute path to address database position
style: move pylint comments near the line causing the issue
fix: use ApplicationBuilder.post_init() to add_commands refactor: rename login to register
| db_path = os.path.join(os.path.dirname(os.path.realpath(__file__)), "db.sqlite3") | ||
| engine = create_engine("sqlite:///" + db_path) | ||
| Session = sessionmaker(engine) |
There was a problem hiding this comment.
Evita sempre di mettere direttive a questo livello, in quanto verranno eseguite nel momento in cui il modulo viene importato. Senza saperlo sto provocando un side-effect (creando un file).
Inoltre rendi piu' delicati i test (cosa succede se voglio usare un altro path invece di db.sqlite?)
There was a problem hiding this comment.
Essenzialmente l'ideale sarebbe rinchiudere tutto dentro una funzione da chiamare al momento del bisogno, possibilmente anche dal main.
Non c'è bisogno di avere una sessione globale: puoi crearne una nuova ogni volta che ti serve, è praticamente equivalente. Leggi questa risposta da uno degli sviluppatori per approfondire.
Se proprio pensi di aver bisogno di uno stato globale, un Singleton potrebbe fare al caso tuo. Almeno puoi controllare il momento in cui viene inizializzato, piuttosto che fare tutto nel momento il cui il modulo viene importato
| HELP_CMD_TEXT = '\n'.join(( | ||
| "📬 /report Fornisce la possibilità di poter inviare una segnalazione agli sviluppatori riguardante qualsiasi disservizio", | ||
| "🔑 /register Permette di effettuare la registrazione al sistema" | ||
| )) |
There was a problem hiding this comment.
Puoi usare una stringa piu' lunga con le triple vigolette o il backslash, non c'e' bisogno di creare una tupla e fare il join.
There was a problem hiding this comment.
Mi sembrava meglio farla cosi' per un discorso di leggibilita' del file in se, pero' se avete fatto sempre con le triple quotes mi adeguo
There was a problem hiding this comment.
Qua ci sono alcune opzioni per ottenere lo stesso risultato. Vedi quali preferisci.
ORIGINAL = "\n".join(
(
"📬 /report Fornisce la possibilità di poter inviare una segnalazione agli sviluppatori riguardante qualsiasi disservizio",
"🔑 /register Permette di effettuare la registrazione al sistema",
)
)
BACKSLASH = "📬 /report Fornisce la possibilità di poter inviare una segnalazione agli sviluppatori riguardante qualsiasi disservizio\n\
🔑 /register Permette di effettuare la registrazione al sistema"
TRIPLE_QUOTE = """📬 /report Fornisce la possibilità di poter inviare una segnalazione agli sviluppatori riguardante qualsiasi disservizio
🔑 /register Permette di effettuare la registrazione al sistema"""
CONCATENING = (
"📬 /report Fornisce la possibilità di poter inviare una segnalazione agli sviluppatori riguardante qualsiasi disservizio\n"
"🔑 /register Permette di effettuare la registrazione al sistema"
)
Closes #3 with the following changes:
python-telegram-botfrom13.6to latest (currently20.6).SQLAlchemy 2.0.23.Something to know:
RuntimeWarningdue toadd_commandsbeing anasyncfunction not beingawait-ed inside themainfunction.data/db/__init_.pythere's a very weird import to create the tables, most likely it can be done in a much better way.