[$45] Edit payments

[maxceem]

Background

At the moment we can only create new payments and cancel whole payments if we need to. But sometimes managers would like to change the amount of payment.

Task

• We already can show the list of all the payments:

  

• Replace Cancel button in this list with Actions dropdown menu with 2 options:

  

  • Cancel Payment- which would do the same as Cancel button does now (option should be disabled for cancelled or in-progress payments)
  • Edit Payment - which would open a modal window to edit payment details (option should be disabled in-progress payments, payment in other statuses can be edited)
  • The actions dropdown is already implemented in https://github.com/topcoder-platform/taas-app see demo page https://platform.topcoder-dev.com/taas/myteams/18102. Please, port it from there. Or reuse it if it was already ported during another task [$80] Additional Payments #82.

• For modal, use the same design as for adding payment: [$80] Additional Payments #82

• We have 2 kind of payments:

  • payment for days which user worked (we pay per day as per member rate)
    • all the payments for days have days greater than 0
  • additional payment - any amount which is additional to the payment for days, and is not counted for days
    • all the additional payments have days === 0

• Edit Payment for days

  

  Inside payment for days we should NOT allow editing Amount we can only edit Days
  - reuse the same field which we use for Working Days
  - the minimal value of days is 1 (disable - button if the value is 1 and show tooltip saying: The payment cannot be done for less than 1 day)

  • the maximum value of days: we cannot pay for more days than user worked this week. So WorkPeriod.daysPaid cannot be more than WorkPeriod.daysWorked. WorkPeriod.daysPaid is calculated as a sum of the days of all the payments of this WorkPeriod not including cancelled or failed payments. Disable the + button if increasing one days would cause WorkPeriod.daysPaid to be more than WorkPeriod.daysWorked, see how we calculate it server-side https://github.com/topcoder-platform/taas-apis/blob/dev/src/services/WorkPeriodPaymentService.js#L285-L291

• Edit Additional Payment

  

  For additional payment days === 0 and we cannot edit that. We can only edit the amount. It should float number greater than 0. And less than 100 000. If we enter wrong amount show a red validation error under the field like saying: Amount should be greater than 0 and less than 100 000. It should look like this https://monosnap.com/file/J4f0foiyUi7FvQPCIZEdFFcfIS4hHi try removing job title here https://platform.topcoder-dev.com/taas/myteams/16873/positions/ecaf2c7f-2b43-4397-bcf5-0ddf38c7f6a6/edit

• If we click Cancel or X just close the modal.

• if we click Update show a loading indicator inside the modal OR just disable the fields and all the buttons while sending request to the server.

  • if the request was successful, show a successful green toastr and close the modal which says, Payment was successfully updated:
    
  • in case of error show error red toastr with error message from the server, but keep the modal open so user can try updating payment again

API

• API has been updated to support editing payments [$100] Edit payments fields topcoder-platform/taas-apis#431
• Use PATCH /work-period-payments/:id endpoint https://topcoder-platform.github.io/taas-apis/#/WorkPeriodPayments/patch_work_period_payments__id_

[sandhiyakavi]

@maxceem

1. In the modal, if we enter floating values (even though less than 100,000) with six digits, it is showing wrong error message "Amount should be greater than 0 and less than 100,000."
   

2. Can we disable Edit payment for Failed status, as it is of no use in editing it right?
   

3. Sometimes if days cannot be increased/decreased because of working days limit, can we disable update button?
   

Other than that, it is working fine as expected

[maxceem]

Sum up:

1. Need to fix.

2. Let's disable "edit" button for "failed" and for "cancelled" payments.
   Note, that for "cancel" button should be only disabled for "cancelled" payments, but still enabled for "failed".

3. Let's disable "update" button if we didn't change anything inside "edit payment" model (days or amount).

@MadOPcode feel free to pick up this issue as per this comment.