-
Notifications
You must be signed in to change notification settings - Fork 27
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
Read HDFC Statements in XLS Format and convert to CSV format #2
Comments
Sample HDFC Statement in XLS Format - https://drive.google.com/open?id=1kG7a6x2k5Hjaj5IF7Vn1QXsT-0AIpk-R |
@MovingToWeb Will start working on this. Planning to create a spring boot project for this. |
@MovingToWeb I have started with implementing a parser module using the POI library. I am considering the following attributes for the CSV file. Please have a look and let me know if this fits the generic CSV format and do we have all required attributes
|
|
@TechSourav The task which I am currently working on is to read the HDFC statement and to convert it to CSV format. Are these two separate tasks or am I missing something here? |
@varunu28 Instead of going entity by entity(e.g. HDFC), should we just have a generic module for CSV format statements and pass each entity into it? So we can make use of this generic module for all the other entities. What do you say? |
@TechSourav We can have a generic format for the CSV but I suppose we need to have different code modules for each of bank statements unless the bank statements in xlsx follow the same format across all the banks. Therefore, the approach which I suggest is to have an interface for the CSV format and pass that as specific code modules to get the required attributes. |
@varunu28 This seems correct. So will you write the interface for the CSV format in this same issue? |
@TechSourav Yes. This is the suggested format for CSV
@MovingToWeb Can you please also provide some input on the format? |
I think, we would just duplicate the data here? each row contain same data except
I think, we should re-think and come with a better approach? |
We can have two different interfaces. One for user details and another for transactions. These two can be mapped based on the Customer ID. |
Sorry guys. Got busy with some personal stuff. How about a Hangout to discuss and close this one? |
@MovingToWeb Sounds great. Let's decide up on a time. |
No description provided.
The text was updated successfully, but these errors were encountered: