-
Notifications
You must be signed in to change notification settings - Fork 5
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
[P/L Addon] Overall P&L Value lower than the Unrealized P&L Value #10
Comments
thanks for reporting, let me check out what would have went wrong. |
I pushed in a potential fix today, please check and let me know. |
Hi,
I have checked it on the wealthica web interface. I still see the bug. I
was wondering do I have to make a new transaction first? Does the new
change apply to historic data as well?
…On Sat, 9 Oct 2021 at 19:44, Elayamani K ***@***.***> wrote:
I pushed in a potential fix today, please check and let me know.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADVMU4ZOSEB6CHRWEGPXDXLUGB5QHANCNFSM5EVUJNQQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Best regards,
Constantin Kwiatkowski
|
@ultimosparc what kind of account is it? I found this issue with one of my crypto, the issue was that I don't consider the Saturday & Sunday values for the PnL which messed up the numbers. |
Hi I am using a tfsa
Elayamani K ***@***.***> schrieb am So. 10. Okt. 2021 um
19:01:
@ultimosparc <https://github.com/ultimosparc> what kind of account is it?
I found this issue with one of my crypto, the issue was that I don't
consider the Saturday & Sunday values for the PnL which messed up the
numbers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADVMU45UR3CZKBKDEIUCEQTUGHBGPANCNFSM5EVUJNQQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Best regards,
Constantin Kwiatkowski
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, I have checked your add-on and the thing is that the Overall P&L Value lower than the Unrealized P&L Value is. Please find attached picture below.
The text was updated successfully, but these errors were encountered: