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.

Sign up for GitHub

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

Jump to bottom

update validator-bal 2.17.0 #329

Merged
merged 1 commit into from
Dec 7, 2023
Merged

update validator-bal 2.17.0 #329

merged 1 commit into from
Dec 7, 2023

Conversation

fufeck
Copy link
Contributor

@fufeck fufeck commented Nov 22, 2023

No description provided.

@nkokla
Copy link
Contributor

nkokla commented Nov 23, 2023

Hello !
Il faudrait qu'on prenne le temps de discuter de cette PR et du validateur BAL.
Il y a quelques points concernant la mise à jours du validateur sur lesquels j'ai un doute. (En cas d'ID incorrect, il devrait y avoir un error plutôt qu'un simple warning et l'absence de numéro sur l'adresse devrait uniquement, au moins en mode relax, renvoyer un warning).
De plus la question se pose sur l'utilité de conserver le validateur sur BAN-Plateforme (Il me semble que les fichiers doivent déjà passer le validateur sur l'API-Depot, du coup c'est un peu redondant).
Bref, à dispo pour en discuter.

@antoineludeau antoineludeau merged commit 504e133 into master Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antoineludeau antoineludeau antoineludeau approved these changes

@nkokla nkokla Awaiting requested review from nkokla

@MaGOs92 MaGOs92 Awaiting requested review from MaGOs92

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fufeck @nkokla @antoineludeau