killbill / killbill-admin-ui

Kill Bill Administrative UI engine
https://killbill.io
MIT License
53 stars 60 forks source link

Making Transfer bundle Policy options consistent with API #320

Open KBbitsP opened 2 years ago

KBbitsP commented 2 years ago

Issue : Currently when a bundle is transferred from one account to another using Kaui, the screen does not give any option to select the transfer policy and uses End_Of_Term as the default value. On the other hand, the same operation when performed via the API gives the option to select out of Start_Of_Term, Immediate OR End_Od_Term policy just like at the time of cancelling.

image

Expected behavior : The behavior and options should be consistent between the admin ui and the API ( https://killbill.github.io/slate/#bundle-transfer-a-bundle-to-another-account ).

KBbitsP commented 2 years ago

This is interlinked to the below issue :

https://github.com/killbill/killbill/issues/1775

kpbacode commented 2 years ago

Screenshot 2022-10-04 at 13 26 15

kpbacode commented 2 years ago

https://github.com/killbill/killbill-admin-ui/pull/321

reshmabidikar commented 2 years ago

Looks good (see screenshot below). A few comments:

  1. Currently, the billing policy defaults to IMMEDIATE. Wondering if we should default it to END_OF_TERM to keep it consistent with the API?
  2. As per @KBbitsP, there is an issue when the START_OF_TERM policy is specified (#1775). So, should the START_OF_TERM policy be disabled in Kaui until this issue is fixed?
image
reshmabidikar commented 2 years ago

Any thoughts @pierre?

pierre commented 2 years ago

Agreed x2!

reshmabidikar commented 2 years ago

FYI @kpbacode.