lenra-io/lenra_components

Is 'utils' belongs to this library ?

Aperta

#7 aperta il 18 ago 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Dart (0 fork)auto 404
help wantedinvalidquestion

Metriche repository

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

Descrizione

There is a folder 'utils' that give some helpers to :

  • parse a color
  • validate textfield input (password/email)

The color parser is not used by the library. It is just exposed as part of the lib. The validator is partially used by the lenra_text_form_field.dart component to check the value of the input based of the type parameter (email or password).

I think we can safely remove the color parser but i'm not sure about the validator :

  • Is it our job to check the value of the input ? If not, we should remove the validator.
  • If it is our job, we should provide a way for the user to completely override the validator (not just add on top of the existing validator).
  • Should we add more types in the futur ? (phone number, address...)

@lenra-io/lenra any thought ?

Guida contributor

Direzione di ricerca
Analizza l'utilizzo dei componenti della cartella utils (parser colore e validatore) all'interno e all'esterno della libreria. Decidi se rimuovere il parser colore, rifattorizzare il validatore per consentire la sovrascrittura completa, o estrarre le utilità in un pacchetto separato. Considera lo scopo della libreria e i piani futuri.
Tech stack
dartflutter
Dominio
frontendmobile
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
DartFlutter
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50