-
Notifications
You must be signed in to change notification settings - Fork 146
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
default value in ImmediateOrigin error is always displayed, regardless of value #1430
Comments
If there's no value in the ImmediateOrigin field then an error will be displayed. Is the field blank? With achcli you can supply a JSON file of the
Or you can disable all validation for a quick sanity check.
|
The field was not blank. There was a valid value in the ImmediateOrigin field, and the error was still displayed. |
Can you share more about the field's value or the full header line with me in private? adam@moov.io |
I am providing a test file for you, along with the options file and command line used for testing.
|
|
Immediate Origin is not restricted to routing numbers, and many banks we deal with require EIN, TID or Company Id in this field. I'm not sure the program should be assuming the field will contain a routing number. |
Even the nacha.org website states, "Often your ODFI will have you insert your company ID in this field." |
I agree that it's very common to not use a routing number. That's why we offer This library aims to implement the Nacha spec, but work for all real world situations. Currently I see the Nacha spec requiring that field is a routing number. |
If the error message is updated to replace the |
Yea we can do that. |
ACH Version
v1.35.1 cli
What were you trying to do?
We were trying to validate an ACH file.
What did you expect to see?
We expected not to have an *ach.FieldError on line 1 because the file input to the program did not have a default value in ImmediateOrigin.
What did you see?
line:1 record:FileHeader *ach.FieldError ImmediateOrigin xxxxxxxxx is a mandatory field and has a default value, did you use the constructor?
How can we reproduce the problem?
Try validating any file. We are unable to avoid the error.
The text was updated successfully, but these errors were encountered: