-
Notifications
You must be signed in to change notification settings - Fork 0
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
Belgium vat validation issue #5
Comments
Is the question if we want the BE VAT input validation without the 0 to fail or if we want the validation to pass but somehow signal that the initial input was invalid? Asking because the latter was suggested here. |
I'm also wondering if Belgium is the only country we change such input? I would suggest having strict validation in any case, without changing the input value. So validating based on formats: Regex would be: Making the following valid: And making the following invalid: And we leave it to the developers to add it before validating if they would like to validate only numbers of vat: This would also mean we don't need to change the input nor alert that an input has changed. |
Like previously mentioned, I've changed Belgium such that the format becomes: "BE" prefix currently needs to be present, but can be changed if that is a use case that is needed. Validating without BE could be achieved by: |
A Belgian company number without BE is indeed a valid KBO number (local company registry), but not a correct VAT number that will need the prefix to be present |
I'm closing this issue for now, feel free to comment if the community has other concerns about this specific case |
See parent/forked repo: se-panfilov/jsvat#104
I ask to the community, is this something we want ?
The text was updated successfully, but these errors were encountered: