Skip to content

refactor(ui-texts, keys): extraer strings estáticos en clases constantes - #98

Open
https-michi wants to merge 4 commits into
mainfrom
feature/micho
Open

refactor(ui-texts, keys): extraer strings estáticos en clases constantes#98
https-michi wants to merge 4 commits into
mainfrom
feature/micho

Conversation

@https-michi

@https-michi https-michi commented Aug 1, 2025

Copy link
Copy Markdown
Collaborator

Extraer strings estáticos en clases constantes

Qué archivos se modificaron

  • Se crearon nuevos módulos en presentation/resources/constants/ para centralizar textos, claves y etiquetas estáticas
  • Se modificaron múltiples archivos en presentation/pages/ y presentacion/widgets/ para reemplazar strings quemados por constantes

Cambios realizados

  • Refactorización general para extraer textos y claves estáticas a clases constantes organizadas por contexto
  • Se crearon archivos por dominio
  • Se actualizaron aproximadamente 50 archivos para usar las nuevas constantes
  • Mejora en la mantenibilidad y legibilidad del código UI

CheckList

  • El código compila correctamente
  • No hay advertencias de Formato de Código (dart format . --output=none --set-exit-if-changed)
  • No hay advertencias del linter (flutter analyze --fatal-warnings)
  • He probado los cambios manualmente
  • La rama está actualizada con main

Notas Adicionales

Esta refactorización sienta la base para mantener un control centralizado de textos y claves reutilizables. Puede ampliarse en el futuro según se añadan más módulos.

Repository owner deleted a comment from https-michi Aug 2, 2025

@RodriHerrada RodriHerrada left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo ok

@RodriHerrada RodriHerrada left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo ok

@AllanSagastegui AllanSagastegui left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resuelve conflictos antes de hacer merge
Conflicto en: lib/presentation/pages/serviexpress.dart

@AllanSagastegui AllanSagastegui added the refactor General code refactor label Aug 2, 2025
@AllanSagastegui

AllanSagastegui commented Aug 2, 2025

Copy link
Copy Markdown
Collaborator

Que buen trabajo , sigue asi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor General code refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants