search

engswee / equalize-cpi-converter

CPI Converter a.k.a. FormatConversionBean for CPI
MIT License
17 stars 10 forks source link

Cpi converter xml to excel (need custom field type date instead of text) #19

Open Prasanthi31 opened 1 month ago

Prasanthi31 commented 1 month ago

Hi Eng,

Thank u so much for ur contribution and xml to excel converter in cpi is working great but I have a requirement where I need to change few columns to date as default excel created is showing general text. I see the same requirement was asked in the forum but didn’t find any enhancement so wondering if u can point to me to he right direction?

thank u!!

engswee commented 1 month ago

Hi @Prasanthi31 Thank you for raising this issue. At the moment, the cells in Excel are created as string/text types. There is no enhancement to handle different cell types. I can't provide a definitive timeline on providing such an enhancement. If this is something needed urgently, you could consider forking the project and making changes to suit your requirement.

Prasanthi31 commented 1 month ago

Hi Eng Swee,

Thank you for your reply. I was able to manage my custom column type format with additional groovy script in my iflow in CPI.

Thanks, Prasanthi

On Mon, Sep 23, 2024 at 3:48 AM Eng Swee Yeoh @.***> wrote:

Hi @Prasanthi31 https://github.com/Prasanthi31 Thank you for raising this issue. At the moment, the cells in Excel are created as string/text types. There is no enhancement to handle different cell types. I can't provide a definitive timeline on providing such an enhancement. If this is something needed urgently, you could consider forking the project and making changes to suit your requirement.

— Reply to this email directly, view it on GitHub https://github.com/engswee/equalize-cpi-converter/issues/19#issuecomment-2367584658, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMMZFVE75TR7J5XMZLPFY5TZX7IVLAVCNFSM6AAAAABOTSALY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRXGU4DINRVHA . You are receiving this because you were mentioned.Message ID: @.***>

engswee commented 1 month ago

Hi @Prasanthi31

Good to hear that you managed to handle your requirement. I'll keep this issue open as I think it's a worthwhile enhancement - hopefully I'll be able to find time to get around to it at some point!

Prasanthi31 commented 1 month ago

Hi Eng Swee,

I never had any experience working on git repo earlier so I was a little hesitant to try. But I will try to take out some time to clone the rep in my local machine and do explore to extend this.

The ConversionExcelOutput class is the one which we need to modify to take specific column numbers which need to be customized for cell type.

On side note: I would like to request you if you are offering any course on API management and advanced event mesh in integration suite? I wanted to enroll in it?

Thanks, Prasanthi

On Mon, Sep 30, 2024 at 1:34 PM Eng Swee Yeoh @.***> wrote:

Hi @Prasanthi31 https://github.com/Prasanthi31

Good to hear that you managed to handle your requirement. I'll keep this issue open as I think it's a worthwhile enhancement - hopefully I'll be able to find time to get around to it at some point!

— Reply to this email directly, view it on GitHub https://github.com/engswee/equalize-cpi-converter/issues/19#issuecomment-2383900386, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMMZFVFIK3GHM3WQCDXXT33ZZGKVBAVCNFSM6AAAAABOTSALY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBTHEYDAMZYGY . You are receiving this because you were mentioned.Message ID: @.***>