Closed rtshiva closed 4 months ago
Many thanks for your contribution, I'll review and and update.
@SudheerNotes , one more change was needed on top of this hence i had closed this PR. you can look at the latest changes in https://github.com/SudheerNotes/cams2csv/compare/main...rtshiva:cams2csv:main
Ah ok...I'll have a look at it this weekend.
here is the new PR with the fixes https://github.com/SudheerNotes/cams2csv/pull/9/files . you can merge if the changes look good.
Fund Name field was containing ISIN, Advisor & Registrar in the earlier commit. This patch has modified the output to show Fund Name separate from ISIN, Advisor & Registrar in their own columns.
Also fixed the regex parsing to ensure that whenever fund name goes to next line it is also considered and included in the final output generated.