search

klequis / private-money-client

5 stars 2 forks source link

Omitted tx are present in tx table #9

Closed klequis closed 3 years ago

klequis commented 3 years ago

Omitted tx should not be retrieved from db.

Check if call to db has been modified for debugging purposes.

klequis commented 3 years ago

Appears the come to omit transactions when db is queried has been removed. Needs some thought.

klequis commented 3 years ago

If the user wants to see omitted transactions it makes sense to show only omitted transactions. Don't see any value in seeing them both at the same time.

  1. User loads app
    • Show omitted is unchecked
    • All transactions with omit=false or no omit property are shown
  2. User clicks Show omitted
    • Show omitted is checked
    • Only transactions with property omit=true are show
  3. User clicks Show omitted
    • Show omitted is unchecked
    • State of UI is same as point 1 above
klequis commented 3 years ago

This feature is implemented.