-
Notifications
You must be signed in to change notification settings - Fork 29
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
National bank code #5
Comments
There's no support for that in the library at this time. At face value, decoding the bank code should be fairly easy. However, doing so opens up the requirement for adding all the other country-specific features in the various IBANs. Most of these exist for a few countries but not for most others, where you'd have to return I'm intentionally keeping the I would accept a separate utility class with static methods that take an |
Thanks a lot for your answer. I'm actually using |
I'm not completely sure about the class name `IBANExtras`, I find it a bit clunky and may rename it before releasing a new version of this library.
@mfirry I gave it a go and implemented the bank code / branch code extraction based on the SWIFT spec. I'm not entirely happy with the class naming, would love to hear if you have something better. The reasoning is this:
Basically, I'm now going to update my test code. If the autocompletion in the various IDE's puts I'm also quite ambivalent to |
@ikanters I know you're using this library – any thoughts on the above? Thanks :). |
I had considered |
Hi is there anyway to add a functionality to discover the National bank code?
The text was updated successfully, but these errors were encountered: