blu3-bird/Shopping-on-tuesday

Replace hard-coded route strings with constants in auth/routes.py

Aperta

#46 aperta il 10 dic 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)CSS (16 fork)auto 404
documentationfrontendhelp wanted🐭good first issue🫎enhancement

Metriche repository

Star
 (5 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

[Refactor] Replace hard-coded route strings with constants in auth/routes.py

Description

The authentication routes file (app/auth/routes.py) contains hard-coded route name strings that should be replaced with constants for better maintainability.

Current Issues

File: app/auth/routes.py

Hard-coded strings found:

  • 'admin.dashboard' (appears 2 times in login function)
  • 'main.index' (appears 1 time in logout function)
  • 'auth.login' (appears 1 time in login function)

Problems

  • Typos only caught at runtime
  • Difficult to refactor if route names change

Reference

  • DEFAULT_AFTER_LOGIN = 'admin.dashboard'
  • DEFAULT_AFTER_LOGOUT = 'main.index'

Labels: enhancement, refactoring, code-quality Scope: app/auth/routes.py

Guida contributor

Direzione di ricerca
Apri app/auth/routes.py e sostituisci le stringhe di route hardcoded 'admin.dashboard', 'main.index' e 'auth.login' con costanti. Cerca un file o modulo che definisce queste costanti (ad esempio, DEFAULT AFTER LOGIN, DEFAULT AFTER LOGOUT) e importale. Testa che le route funzionino ancora dopo la modifica.
Tech stack
pythonflask
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonFlaskGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90