-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
feature/drip transaction #1963
base: main
Are you sure you want to change the base?
feature/drip transaction #1963
Conversation
@dtslvr can you please review this? The only thing I tested was adding a "DRIP" transactoin both manually and by JSON import and I see that the transaction is added as expected and it also appears in the "Dividend Timeline". |
f1354f3
to
b5c299a
Compare
Hi @raskanskyz Thanks for your suggestion. I have finally found some time to review your PR. I would like to understand exactly why a new |
Hey @dtslvr, |
Maybe you can create a test scenario @raskanskyz here to verify? Something like |
Hello, I had some comments about DRIP transaction, as example : very close to Merge&Split transaction : Nice and promising application by the way, congrats 👍 |
resolves #1618