search

ananthakumaran / paisa

Paisa – Personal Finance Manager. https://paisa.fyi demo: https://demo.paisa.fyi
https://paisa.fyi
GNU Affero General Public License v3.0
2.24k stars 111 forks source link

Error while importing the password protected XLSX files. #156

Closed swarajsomala closed 5 months ago

swarajsomala commented 5 months ago

Describe your Environment OS: Windows 11 Paisa Version: 0.6.3 App Variant: CLI

Describe the bug I encountered an issue when trying to import a XLSX file into Paisa after Paytm introduced a new password protection for its statements. The following error is being thrown:

image

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Ledger->Import'
  2. Click on 'Drag and Drop'
  3. Choose the XLSX file.

Expected behavior Paisa should show a popup so user can enter the password for the XLSX file.

Screenshots image

ananthakumaran commented 5 months ago

@swarajsomala can you check the file extension? As I mentioned in the discussion thread, CSV file is just ascii characters and can't be password protected. Can you share the file extension? Windows explorer might be hiding the extension, you can use a powershell or other terminal to view it.

swarajsomala commented 5 months ago

@ananthakumaran , I am sorry it’s XlSX file not csv, updated the relevant details.

ananthakumaran commented 5 months ago

@swarajsomala I have pushed some changes to https://demo.paisa.fyi, can you try if that works for you?

swarajsomala commented 5 months ago

@ananthakumaran , Tested its working fine but I can see the password when I enter. Can you hide it?

image

ananthakumaran commented 5 months ago

Can you hide it?

I am using the javascript prompt which doesn't allow me to hide the input. I will later refactor this and use a custom modal (which is more work because the prompt is synchronous, whereas the modal will be async and need to change at different places).

swarajsomala commented 5 months ago

Sure it’s good to have feature. Implement whenever you have some bandwidth, Thanks for adding the support.

ananthakumaran commented 5 months ago

This is now available on 0.6.5