abhirajadhikary06 / abhirajadhikary06/openfeedback

[Bug] Refactor ```auth.py ``` to use SQLAlchemy ORM and fix Circular Imports.

Ouverte
#36 1 commentaire 0 réactions 1 personne assignée Réclamée par @Thedrogon Voir sur GitHub
apertre3.0 medium
Langage dominant
HTML
Étoiles
8
Forks
15
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Currently, the application mixes two different methods of database access:

```app.py``` uses **Flask-SQLAlchemy.**

```auth.py``` uses raw sqlite3 connections.

Problems this causes:

Database Locking: Mixing raw connections with ORM sessions on SQLite can lead to "database is locked" errors and potential corruption.

Maintenance: We are defining the User schema in multiple places (SQL definitions in auth.py vs Model definitions in app.py).

Security: Raw SQL queries are more prone to errors than ORM queries.

@abhirajadhikary06

well I already have solution for this problem , would love to initiate.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.