-
Notifications
You must be signed in to change notification settings - Fork 559
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
Adds FFaker::Bank.routing_number and accounting_number #543
Conversation
ad21fae
to
0b31036
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not against such methods.
But, we probably should think about US
suffix somewhere (in the module name?), if it's specific to a country.
0b31036
to
8ea8bc1
Compare
Done. |
7345443
to
42297dd
Compare
b8d1ac1
to
3ce0fd1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
3ce0fd1
to
cebd12c
Compare
cebd12c
to
c6730be
Compare
@marocchino thank you for the approval. This PR has been in a green state for a little over 7 days with no new comments. I would love to see it merged this week. |
You've waited too long. I think this will be okay. |
This adds
FFaker::Bank.routing_number and accounting_number
For our use case, we're looking for realistic looking US Bank account numbers and routing numbers.
Accounting numbers can very between 9 and 17 digits (Reference) Routing numbers have a checksum calculation on the last digit.