Remove suggestions because suggest=True adds list as metadata which is invalid about smart_importer

[yagebu]

This intended to be used in plugins/other importers/as Fava input suggestions (beancount/fava#801, still TODO).

[tbm]

Would it make sense to propose a list of strings to be allowed for metadata in beancount?

(I'd actually have use cases for that.)

[yagebu]

Would it make sense to propose a list of strings to be allowed for metadata in beancount?

I'd be against that - another case to deal with in all places in Fava and Beancount where metadata is handled. Also, for any specific use case a quick "encode" as one space-separated string is probably enough - and if it isn't, maybe you're trying to stuff something to complicated into the metadata (and encoding to JSON should do it in that case)

[tbm]

You're making some good points.

Anyway, please leave this issue open as a documentation issue for smart_importer.

[johannesjh]

I closed beancount/fava#801 because I don't really see the need for it. Fava's existing suggestions work sufficiently well in my experience.

Shall we remove smart_importer's suggestion feature?

[yagebu]

Shall we remove smart_importer's suggestion feature?

Yep, sounds good to me

[tarioch]

Agree as well, for me works perfectly either the defaulting from smart importer works (which is in about 95% of my cases) or otherwise it's easy to adjust to what I need.