Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: tax_code or incomplete data formal validator #15

Closed
wants to merge 2 commits into from

Conversation

baraidrissa
Copy link
Contributor

@baraidrissa baraidrissa commented Aug 4, 2023

Ciao @matteoredz,
spero di trovarti bene.

Usando la gemma abbiamo trovato che sarebbe interessante dare la possibilità sul metodo ItaxCode.valid? che accetti anche solo un argomento (il CF), o un hash difettivo (solo alcuni fra nome, cognome, genere, luogo e data di nascita) e verifichi la correttezza formale limitatamente ai dati forniti.

Grazie!

@matteoredz
Copy link
Owner

matteoredz commented Aug 30, 2023

Ciao @baraidrissa, grazie per utilizzare questa gemma e per lo spunto di miglioramento.

Ho creato la PR #26 che implementa in maniera un po' più solida quanto richiesto.

Ho preferito lasciar l'identificazione di dati incompleti come dettaglio interno, in modo da non dover modificare la firma del metodo ItaxCode.valid? ma modificando quella l'initializer di ItaxCode::Validator e la firma del suo metodo d'istanza #valid?. Pur trattandosi di una classe ad uso "interno" preferisco rilasciare questa modifica con la prossima major "1.X.X".

Fammi sapere cosa ne pensi.

@baraidrissa
Copy link
Contributor Author

Ciao @matteoredz ,
innanzitutto grazie del gentile riscontro in merito.

Ho appena fatto delle prove sul branch di riferimento, e ti direi che è esattamente ciò che ci serviva.

Nell'attesa del tuo rilascio,
ti auguro buona giornata!!

@matteoredz matteoredz closed this Aug 30, 2023
@matteoredz
Copy link
Owner

matteoredz commented Aug 30, 2023

@baraidrissa la release 1.0.0 è stata mergiata! Enjoy 😄

@baraidrissa
Copy link
Contributor Author

@baraidrissa la release 1.0.0 è stata mergiata! Enjoy 😄

Grazie!

@baraidrissa baraidrissa deleted the develop branch August 31, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants