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

Downgrade Plan #1783

Closed
FSM1 opened this issue Nov 26, 2021 · 11 comments · Fixed by #1804
Closed

Downgrade Plan #1783

FSM1 opened this issue Nov 26, 2021 · 11 comments · Fixed by #1804
Labels
Epic: Billing Added to issues to signal that it is part of an epic Priority: P1 High Added to issues relating to a high severity bugs.

Comments

@FSM1
Copy link
Contributor

FSM1 commented Nov 26, 2021

Overall flow as described in Figma:

If user is over the limit of the desired plan (is_update_allowed is false) then offer users to remove files until they fit in to the target plan.
image

If the user is able to downgrade to the desired plan:
image

@FSM1 FSM1 added Planning: Needs Clarification 🔍 Added to issues relating to a low severity bugs. Planning: Needs estimation Priority: P1 High Added to issues relating to a high severity bugs. labels Nov 26, 2021
@Tbaut
Copy link
Collaborator

Tbaut commented Nov 29, 2021

Agreed with the comments offline that the flow is overly complex, and the users could be informed about the amount of data that should be deleted and delete it themselves.

@FSM1
Copy link
Contributor Author

FSM1 commented Dec 1, 2021

Agreed with the comments offline that the flow is overly complex, and the users could be informed about the amount of data that should be deleted and delete it themselves.

Updated with the latest designs

@FSM1
Copy link
Contributor Author

FSM1 commented Dec 1, 2021

Hey team! Please add your planning poker estimate with ZenHub @tanmoyAtb @Tbaut

@FSM1 FSM1 added Epic: Billing Added to issues to signal that it is part of an epic and removed Planning: Needs Clarification 🔍 Added to issues relating to a low severity bugs. Planning: Needs estimation labels Dec 1, 2021
@Tbaut Tbaut self-assigned this Dec 2, 2021
@Tbaut
Copy link
Collaborator

Tbaut commented Dec 5, 2021

Updated OP with the latest design

@Tbaut Tbaut mentioned this issue Dec 5, 2021
4 tasks
@Tbaut Tbaut removed their assignment Dec 6, 2021
@Tbaut
Copy link
Collaborator

Tbaut commented Dec 6, 2021

Removing myself from this issue for now because some paths are more involved than expected and actually require more info from the api.

So let's say if a user is on a plan A($100 monthly) and wants to migrate to plan B($30 monthly) in the middle of the month. In this case he/she has $50 worth of service credit left.

So we have following cases in plan updates:

  1. User downgrades/upgrade with payment method remains unchanged in case of stripe
  • We support proration. Based on the above example, customer doesn't have to pay any additional money.
  • We support credit. Based on the above example, $20 will be added to stripe credit which will be utilized on the next billing.
    --> we need to tell the user that their downgrade will give them $20 on any following bill.
  1. User downgrades/upgrade with payment method remains unchanged in of crypto
  • We support proration. Based on the example, customer doesn't have to pay any additional money.
  • We don't support credit. We won't be adding $20 to credit.
    --> We need to tell the user that starting from next bill, they will need to pay the full monthly rate.
  1. User downgrades/upgrade along with payment method updated
  • We don't support proration. Based on the above example, customer have to pay full amount i.e. $ 30
  • We don't support credit. We won't be adding $20 to credit.
    --> We need to warn the user that due to payment method change, they will need to pay the monthly rate right away.

Similarly, if a user moves from a monthly paid plan to a yearly paid plan, we'll need to get the calculation regarding what users have already paid, and what is missing.

@serenaho Could we update the Figma flow to let users select their payment method and see the error/info?

@serenaho
Copy link
Collaborator

serenaho commented Dec 6, 2021

@Tbaut For sure, I'll update the flow in Figma to reflect these changes!

@serenaho
Copy link
Collaborator

serenaho commented Dec 7, 2021

Hey @Tbaut, I have a question about the second option in the case of crypto payments you posted above.

Since we don't allow for monthly payments for crypto, if a user downgrades from plan A to plan B then we won't need to tell them to pay the full monthly rate starting from their next bill, right?

Instead, we'll say that they will need to pay again at the end of their pro-rated period. Is this correct?

@Tbaut
Copy link
Collaborator

Tbaut commented Dec 7, 2021

Good point, tbf I'm not sure we don't allow monthly crypto payment, it's just that it doesn't make any sense for our user (for us it doesn't matter). But if someone wants to pay 10$+ in fees for something that costs 10$, I don't think we'll prevent them explicitly.

@serenaho
Copy link
Collaborator

serenaho commented Dec 7, 2021

Right, although since we've disabled the crypto payment option for when the user selects monthly billing, I'm assuming it probably makes sense to do this:

Instead, we'll say that they will need to pay again at the end of their pro-rated period.

@Tbaut
Copy link
Collaborator

Tbaut commented Dec 8, 2021

We haven't disabled the crypto payment for monthly subscription. We havent' developed the crypto payment. I really think the users should decide what's better for them. If them really really want it, why not :D

In any case the api should be designed in such a way that the UI can handle these messages in a generic way. Something like:

  • Is the current payment a reduction (because of pro-rata) if yes show something like "Due to your previous credit, you only need to pay X"
  • Is that credit left: we show "Your next bill will be reduced by X thanks to accumulated credits"

^ these are all very badly phrased suggestions :D

@FSM1
Copy link
Contributor Author

FSM1 commented Dec 8, 2021

We haven't disabled the crypto payment for monthly subscription. We haven't developed the crypto payment. I really think the users should decide what's better for them. If them really really want it, why not :D

There were a number of reasons to do this, but it boils down to giving us more time to refine things such as notifications for billing which will be necessary to have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: Billing Added to issues to signal that it is part of an epic Priority: P1 High Added to issues relating to a high severity bugs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants