beancount / smart_importer

Augment Beancount importers with machine learning functionality.
MIT License
248 stars 29 forks source link

disable suggestions by default? #50

Closed johannesjh closed 6 years ago

johannesjh commented 6 years ago

@tarioch, in CC @yagebu @aumayr: shall we disable suggestions of accounts and payees by default? fava does not handle them well (yet), see https://github.com/beancount/fava/issues/801

tarioch commented 6 years ago

yes, makes sense (that's what I do currently for all my stuff, so having this disabled by default makes sense)

johannesjh commented 6 years ago

ok, thank you for your feedback.

implemented in pull request #52, could you review and merge please? ...it was a rather simple fix, just had to adapt two additional lines in the test cases.