spent icon indicating copy to clipboard operation
spent copied to clipboard

Read HDFC Statements in XLS Format and convert to CSV format

Open MovingToWeb opened this issue 6 years ago • 13 comments

MovingToWeb avatar May 23 '19 17:05 MovingToWeb

Sample HDFC Statement in XLS Format - https://drive.google.com/open?id=1kG7a6x2k5Hjaj5IF7Vn1QXsT-0AIpk-R

MovingToWeb avatar Jun 29 '19 18:06 MovingToWeb

@MovingToWeb Will start working on this. Planning to create a spring boot project for this.

varunu28 avatar Jun 30 '19 18:06 varunu28

@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

Bank Name, Branch Address, User Name, User Address, Cust Id, Statement Start Date, Statement End Date, Account Number, Date, Transaction, Ref No, Date, Withdraw Amount, Deposit Amount, Closing Balance

varunu28 avatar Jul 03 '19 17:07 varunu28

@varunu28 You may want to take the other task which says about "generic CSV model". So we can leave this task for the HDFC specific statement. Just to make sure we track it for a correct task.

TechSourav avatar Jul 07 '19 14:07 TechSourav

@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 avatar Jul 07 '19 14:07 varunu28

@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 avatar Jul 07 '19 14:07 TechSourav

@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 avatar Jul 07 '19 14:07 varunu28

@varunu28 This seems correct. So will you write the interface for the CSV format in this same issue?

TechSourav avatar Jul 07 '19 14:07 TechSourav

@TechSourav Yes. This is the suggested format for CSV

Bank Name, Branch Address, User Name, User Address, Cust Id, Statement Start Date, Statement End Date, Account Number, Date, Transaction, Ref No, Date, Withdraw Amount, Deposit Amount, Closing Balance

@MovingToWeb Can you please also provide some input on the format?

varunu28 avatar Jul 07 '19 17:07 varunu28

@TechSourav Yes. This is the suggested format for CSV

Bank Name, Branch Address, User Name, User Address, Cust Id, Statement Start Date, Statement End Date, Account Number, Date, Transaction, Ref No, Date, Withdraw Amount, Deposit Amount, Closing Balance

@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

Date, Transaction, Ref No, Date, Withdraw Amount, Deposit Amount, Closing Balance

I think, we should re-think and come with a better approach?

anilkumarkatta204 avatar Jul 07 '19 19:07 anilkumarkatta204

@TechSourav Yes. This is the suggested format for CSV

Bank Name, Branch Address, User Name, User Address, Cust Id, Statement Start Date, Statement End Date, Account Number, Date, Transaction, Ref No, Date, Withdraw Amount, Deposit Amount, Closing Balance

@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

Date, Transaction, Ref No, Date, Withdraw Amount, Deposit Amount, Closing Balance

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.

varunu28 avatar Jul 08 '19 00:07 varunu28

Sorry guys. Got busy with some personal stuff. How about a Hangout to discuss and close this one? Saturday sounds fine?

MovingToWeb avatar Jul 10 '19 16:07 MovingToWeb

@MovingToWeb Sounds great. Let's decide up on a time.

varunu28 avatar Jul 11 '19 04:07 varunu28