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

Revert "fix: use SupporterPlusV2 Zuora catalogue plans" #280

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

jamesgorrie
Copy link
Contributor

@jamesgorrie jamesgorrie commented Feb 8, 2024

Reverts #279

I completely forgot - this worked on PROD. Now it doesn't.

The FrontendId in Zuora for PROD is Yearly | Monthly. In CODE it's YearlyV2 | MonthlyV2.

My suggestion is that we revert this change, and change CODE Zuora source data to reflect PROD.

An alternative would be to change the FrontendId in PROD to the V2 version, but I am unsure of the downstream knockon effects.

Copy link
Member

@johnduffell johnduffell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good on the revert - actually I was thinking at the time maybe it would make more sense to change the IDs, but I was worried it might break other things. Clearly it doesn't! I assume there is no duplicate in PROD, the old one has been changed to something else? Otherwise it might get non deterministic.

@jamesgorrie
Copy link
Contributor Author

jamesgorrie commented Feb 8, 2024

I assume there is no duplicate in PROD, the old one has been changed to something else?

Great shout - they are actually the same in PROD - so we are getting lucky I think. Let's have a chat about how to deprecate these properly. Assuming the desire is to remove the references all together.

@jamesgorrie jamesgorrie merged commit 6b1d44b into main Feb 8, 2024
2 checks passed
@jamesgorrie jamesgorrie deleted the revert-279-support-plus-v2 branch February 8, 2024 14:27
@jamesgorrie
Copy link
Contributor Author

I've swapped the older products to MonthyV1 and YearlyV1

@johnduffell
Copy link
Member

sounds good, either that or blank them out would probably be as good.

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

Successfully merging this pull request may close these issues.

2 participants