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 support for Beancount v3 #1859

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

yagebu
Copy link
Member

@yagebu yagebu commented Sep 15, 2024

This uses beanquery to replace the functionality in beancount.query,
which works with both Beancount v2 and v3 - however there are some
differences on columns in beanquery compared to beancount.query.

For importers, this sticks to beancount.ingest on v2 and used beangulp
for v3. It still expectes importers to conform to the v2 importer
protocol for now. Due to changes related to duplicate detection,
duplicate detection is not automatically done by Fava here but can still
be manually specified as hooks.

@yagebu yagebu force-pushed the beancount-v3 branch 5 times, most recently from a5860b0 to 3c7495f Compare September 15, 2024 08:30
@yagebu yagebu mentioned this pull request Sep 15, 2024
2 tasks
@yagebu yagebu force-pushed the beancount-v3 branch 3 times, most recently from 00a01fd to db936d9 Compare September 15, 2024 14:07
This uses beanquery to replace the functionality in beancount.query,
which works with both Beancount v2 and v3 - however there are some
differences on columns in beanquery compared to beancount.query.

For importers, this sticks to beancount.ingest on v2 and used beangulp
for v3. It still expectes importers to conform to the v2 importer
protocol for now. Due to changes related to duplicate detection,
duplicate detection is not automatically done by Fava here but can still
be manually specified as hooks.
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.

1 participant