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

Subscription renewal after expiration extends from the past, instead of $now #172

Open
eigood opened this issue Aug 18, 2021 · 3 comments
Open

Comments

@eigood
Copy link
Contributor

eigood commented Aug 18, 2021

In fulfill#ProductSubscriptionResource, there is a comment determine fromDate (either last Subscription.thruDate if in future, or nowTimestamp; however, that 'if in future' bit is never actually tested for in the following logic.

What has occurred, is that my account in our system had an activate subscription that ran up to August 1st. But then, we implemented a change to our database process, and now local development systems no longer have correct production credit card processor settings. So, the nightly process that renews the monthly subscription failed(this is actually fine), so as of right now(August 18th), I do not have an active subscription.

I then attempted to create a new subscription(using prorated days, 14, to round out to the end of the month). The linked service above then attempted to extend the subscription from August 1, through August 15th, instead of $now to August 31.

@eigood
Copy link
Contributor Author

eigood commented Aug 18, 2021

@eigood
Copy link
Contributor Author

eigood commented Aug 18, 2021

That uploaded file fixed this issue for me.

@pythys
Copy link

pythys commented Mar 3, 2022

I think this is already fixed and so it might be better to close this ticket. Although this caused another problem which we'll report in another ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants