search

zhang-accounting / zhang

账 - a plain text double-accounting tool which is compatible with beancount but more powerful
https://zhang-accounting.kilerd.me/
Apache License 2.0
177 stars 10 forks source link

use graphql subscription to update the query on large ledger #60

Closed Kilerd closed 1 year ago

Kilerd commented 2 years ago

when testing the beancount example file, it need 2 seconds to load and analysis the accounting file so if we trigger the reload process on any modification, it would case a bad user experience, so maybe need to use the graphql subscription to notify frontend to refresh the data

Kilerd commented 2 years ago

after fixing the performance issue of pest, the parsing speed is still fast enough stand at hundreds millisecond, so it's not so urgent to fix this issue. but it'd better have the kind of feature, cuz when the backend's data source's changed, the frontend wont know anything currently. the backend should notify the frontend to update/refetch the data at the same time.

Kilerd commented 1 year ago

no needed, cuz we have deprecated graphql