Skip to content
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

Add payee transformation function #405

Conversation

ThomDietrich
Copy link
Contributor

@ThomDietrich ThomDietrich commented Nov 16, 2021

Resolves #404

@ThomDietrich
Copy link
Contributor Author

This did a good job importing-exporting 280 transactions

@raiviskrumins
Copy link

This is crucial functionality for me, thank you for this.
As an additional "feature", it would be awesome if this supported regex
One of our retailers is "Rimi", and depending on store location I get different Payees:

[Payee]
; Rimi
RIMI HM MOLS\KRASTA IELA 46\RIGA\LV-1003      LVA = Rimi
RIMI CHM VALDEMARA (RI\KR. VALDEMARA IELA 112\RIGA\LV-1013      LVA = Rimi
RIMI LHM ALFA (RIGA)\BRIVIBAS GATVE 372\RIGA\LV-1006      LVA = Rimi
RIMI CHM Saga (Stopini\Bikeru iela 4\DREILINI\LV-2130      LVA = Rimi
...and many more

if I could use regex like this, it would match all of these

[Payee]
; Rimi
^RIMI.* = Rimi

@ThomDietrich
Copy link
Contributor Author

Hey @raiviskrumins,
that makes sense to me. The solution would probably be simple, however it is not possible atm, because the configuration file with its ini format is quite restricted. Regex expressions as the key will not be parsed correctly. @nocalla another reason why to switch to yaml for all configuration.

@nocalla
Copy link
Member

nocalla commented Nov 29, 2021

Regex expressions as the key will not be parsed correctly. @nocalla another reason why to switch to yaml for all configuration.

Regex is already supported in the configuration file for file names.

@nocalla nocalla changed the base branch from develop to refactor-and-pandas-401 December 19, 2021 16:36
@nocalla nocalla deleted the branch bank2ynab:refactor-and-pandas-401 January 4, 2022 13:32
@nocalla nocalla closed this Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transformation map for payee names
3 participants