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

Generator: Add bikeNumberValidation and fix carNumberValidation #615

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

samuel-duhaime
Copy link
Collaborator

Pull request

Description

Generator: Add bikeNumberValidation and fix carNumberValidation

@samuel-duhaime samuel-duhaime added the generator About Evolution-Generator packages label Jul 26, 2024
@samuel-duhaime samuel-duhaime requested a review from tahini July 29, 2024 13:00
Copy link
Contributor

@tahini tahini left a comment

Choose a reason for hiding this comment

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

Ce serait bien de mettre ces validations plutôt dans evolution-common si possible? Je ne pense pas que ça nécessite quoi que ce soit frontend? Peut-être des trucs users. En tout, cas, il faudrait voir à emmener ça dans un package d'evolution, pour qu'une enquête qui n'utlise pas le génrérateur puisse en profiter

@samuel-duhaime
Copy link
Collaborator Author

Ce serait bien de mettre ces validations plutôt dans evolution-common si possible? Je ne pense pas que ça nécessite quoi que ce soit frontend? Peut-être des trucs users. En tout, cas, il faudrait voir à emmener ça dans un package d'evolution, pour qu'une enquête qui n'utlise pas le génrérateur puisse en profiter

Oui, je suis d'accord, j'ai donné mon avis à ce propos dans #532

@samuel-duhaime samuel-duhaime merged commit c529fa4 into main Jul 29, 2024
4 checks passed
@samuel-duhaime samuel-duhaime deleted the addBikeNumberValidation branch July 29, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
generator About Evolution-Generator packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants