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

Edit Stock Investment dialog errors #1751

Closed
slodki opened this issue Jul 31, 2018 · 0 comments
Closed

Edit Stock Investment dialog errors #1751

slodki opened this issue Jul 31, 2018 · 0 comments
Assignees
Labels

Comments

@slodki
Copy link
Contributor

slodki commented Jul 31, 2018

  1. Create investment account

  2. Add stock investment with any required parameters

  3. Open Edit Stock Investment dialog
    mmex2

  4. Use download stock price history button or manually add some historical prices

  5. Select historical price on the list

  6. Current Price value is changed
    -> this field contains historical price now, not current as labeled!

  7. Current Value is not changed
    -> inconsistent with Current Price two lines above

  8. Add manually historical price using Current Price (why user should enter historical price into current price field???), Price Date and Add button

  9. Current Value is recalculated using added historical price
    -> value presented here is not current value!

  10. Delete latest/top-most price from list

  11. Current Value is not updated
    -> presented value is based on non-existing price, will change on dialog reopening

Proposal:

  • Move price and date fields to the right pane between the price list and buttons, renaming to simple Date and Price
  • Fix Current Value calculation - use latest price always with special solution (?) for empty price list case

Version: 1.4.0-alpha.4 • db 13 (aes128cbc) • git 08ed887 (2018-07-23) fix-1738
Libs: wxWidgets 3.1.1 (wxGTK 2.24) • wxSQLite3 4.0.4 (SQLite 3.24.0) • Mongoose 6.11 • RapidJSON 1.1.0 • Lua 5.3.3 • libcurl 7.58.0 • OpenSSL 1.1.0g • zlib 1.2.11 • libidn2 2.0.4 • libpsl 0.19.1 • ( libidn2 2.0.4) • nghttp2 1.30.0 • librtmp 2.3
Build: Jul 29 2018 23:38:22 • CMake 3.10.2 • GNU Make 4.1 • gettext 0.19.8.1 • GCC 7.3.0 • Ubuntu 18.04 'bionic'
OS: Ubuntu 18.04.1 LTS "bionic" • Linux 4.15.0-29-generic x86_64 • KDE Polish (UTF-8) • 1280x1024 24bit 96x96ppi
@slodki slodki added the bug label Jul 31, 2018
@slodki slodki added this to backlog in User Experience Aug 19, 2018
@vomikan vomikan self-assigned this May 2, 2019
vomikan added a commit that referenced this issue May 11, 2019
@vomikan vomikan closed this as completed May 11, 2019
vomikan added a commit that referenced this issue May 15, 2019
vomikan added a commit that referenced this issue May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

2 participants