Adzacam commited on
Commit
76765ac
·
1 Parent(s): 68fa78f

refactor: rename analista_datos role to analista_datos_marketing and remove automatic table creation

Browse files
Files changed (2) hide show
  1. app.py +3 -3
  2. database.py +3 -1
app.py CHANGED
@@ -157,7 +157,7 @@ def login_for_access_token(form_data: OAuth2PasswordRequestForm = Depends(), db:
157
  test_users = {
158
  "directivo@giragroup.com": {"password": "Directivo@123", "role": "comite_directivo"},
159
  "academico@giragroup.com": {"password": "Academico@123", "role": "coordinador_academico"},
160
- "datos@giragroup.com": {"password": "Datos@123", "role": "analista_datos"},
161
  "admin@giragroup.com": {"password": "Admin@123", "role": "admin"}
162
  }
163
  if form_data.username in test_users and form_data.password == test_users[form_data.username]["password"]:
@@ -529,7 +529,7 @@ class FinancePayload(BaseModel):
529
 
530
  @app.post("/api/v1/ingesta/financiera", status_code=status.HTTP_201_CREATED)
531
  def procesar_registro_financiero(payload: FinancePayload, db: Session = Depends(get_db), current_user: Users = Depends(get_current_user)):
532
- if current_user.role not in ["analista_datos", "admin"]:
533
  raise HTTPException(status_code=403, detail="Acceso denegado: Se requiere rol de Analista de Datos.")
534
  try:
535
  nuevo_hecho = FactSituacionFinanciera(
@@ -549,7 +549,7 @@ def procesar_registro_financiero(payload: FinancePayload, db: Session = Depends(
549
 
550
  @app.post("/api/v1/ingesta/financiera/bulk", status_code=status.HTTP_201_CREATED)
551
  def procesar_lote_financiero(payloads: List[FinancePayload], db: Session = Depends(get_db), current_user: Users = Depends(get_current_user)):
552
- if current_user.role not in ["analista_datos", "admin"]:
553
  raise HTTPException(status_code=403, detail="Acceso denegado: Se requiere rol de Analista de Datos.")
554
  try:
555
  for payload in payloads:
 
157
  test_users = {
158
  "directivo@giragroup.com": {"password": "Directivo@123", "role": "comite_directivo"},
159
  "academico@giragroup.com": {"password": "Academico@123", "role": "coordinador_academico"},
160
+ "datos@giragroup.com": {"password": "Datos@123", "role": "analista_datos_marketing"},
161
  "admin@giragroup.com": {"password": "Admin@123", "role": "admin"}
162
  }
163
  if form_data.username in test_users and form_data.password == test_users[form_data.username]["password"]:
 
529
 
530
  @app.post("/api/v1/ingesta/financiera", status_code=status.HTTP_201_CREATED)
531
  def procesar_registro_financiero(payload: FinancePayload, db: Session = Depends(get_db), current_user: Users = Depends(get_current_user)):
532
+ if current_user.role not in ["analista_datos_marketing", "admin"]:
533
  raise HTTPException(status_code=403, detail="Acceso denegado: Se requiere rol de Analista de Datos.")
534
  try:
535
  nuevo_hecho = FactSituacionFinanciera(
 
549
 
550
  @app.post("/api/v1/ingesta/financiera/bulk", status_code=status.HTTP_201_CREATED)
551
  def procesar_lote_financiero(payloads: List[FinancePayload], db: Session = Depends(get_db), current_user: Users = Depends(get_current_user)):
552
+ if current_user.role not in ["analista_datos_marketing", "admin"]:
553
  raise HTTPException(status_code=403, detail="Acceso denegado: Se requiere rol de Analista de Datos.")
554
  try:
555
  for payload in payloads:
database.py CHANGED
@@ -36,7 +36,6 @@ engine = create_engine(
36
  SessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
37
 
38
  Base = declarative_base()
39
- Base.metadata.create_all(bind=engine)
40
 
41
 
42
  # =============================================================================
@@ -146,3 +145,6 @@ def get_db():
146
  yield db
147
  finally:
148
  db.close()
 
 
 
 
36
  SessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
37
 
38
  Base = declarative_base()
 
39
 
40
 
41
  # =============================================================================
 
145
  yield db
146
  finally:
147
  db.close()
148
+
149
+ # Database tables creation is handled manually in Supabase.
150
+ # Base.metadata.create_all(bind=engine) has been removed.