klequis / private-money-client

5 stars 2 forks source link

Using request status has hidden requirements #73

Closed klequis closed 3 years ago

klequis commented 3 years ago

Problem: Currently, to add a new slice with fetch functionality the developer needs to be aware of several areas of code to make changes. It is unlikely they would remember.

Goal: Reduce the number of places changes need to be made when implementing a new slice.

  1. Send selectRequestStatus slice names instead of names. For example, txFetch like txFetchRequestStatus.
  2. Inside selectRequestStatus use the slice names instead of longer fetch status constants.
  3. Delete the longer fetch status constants.
klequis commented 3 years ago

done