search

emdeh / pdf-document-processor

0 stars 0 forks source link

Reference to field in process_transactions() function not dynamic #5

Closed emdeh closed 3 months ago

emdeh commented 4 months ago

https://github.com/emdeh/pdf-document-processor/blob/cb5414a78a2193739ad979a60229cb0f8fb3e90e/src/csv_utils.py#L28

'Transactions' in document.fields: is accountTransactions for one type and Transactions for another. Need to make it dynamic.

emdeh commented 3 months ago

Updated field table name at source to Transactions so consistent across existing models. Updated doco to ensure transaction table always called TransactionsTable

emdeh commented 3 months ago

retraining one model to test. Confirmed the other existing model works.