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

Improve portfolio UI view #163

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Improve portfolio UI view #163

merged 2 commits into from
Oct 6, 2023

Conversation

oxisto
Copy link
Owner

@oxisto oxisto commented Oct 6, 2023

  • Properly rendering dates
  • Adding performance gain/loss on portfolio card

* Properly rendering dates
* Adding performance gain/loss on portfolio card
@codecov
Copy link

codecov bot commented Oct 6, 2023

Codecov Report

Merging #163 (a199252) into main (b6b706a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #163   +/-   ##
=======================================
  Coverage   86.22%   86.22%           
=======================================
  Files          19       19           
  Lines         900      900           
=======================================
  Hits          776      776           
  Misses         87       87           
  Partials       37       37           
Flag Coverage Δ
unittests 86.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
service/portfolio/snapshot.go 95.00% <ø> (ø)

@oxisto oxisto merged commit cb342c5 into main Oct 6, 2023
4 checks passed
@oxisto oxisto deleted the improve-portfolio-ui branch October 6, 2023 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant