-
Notifications
You must be signed in to change notification settings - Fork 121
Remove IAP code and usage #16128
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
Remove IAP code and usage #16128
Conversation
Generated by 🚫 Danger |
|
|
||
| /// WPCOM Payment Endpoints | ||
| /// | ||
| // periphery:ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Temporary periphery ignores so I don't have to update PaymentRemote, is possible that we can remove these as well as part of 12401-gh continuation, since are related to domain purchases.
|
|
|
@itsmeichigo looping you in as well for awareness |
| switch rawValue { | ||
| case "woo_notification_store_creation_ready": | ||
| self = .storeCreationCompleteNotification | ||
| case "woo_hardcoded_plan_upgrade_details_milestone_1_are_accurate": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll also have to clean the flag from https://public-api.wordpress.com/wpcom/v2/mobile/feature-flags. I haven't tackled this as part of the PR.
|
👋 @joshheald Feel free to ignore this review during HACK week and team meetup, I'll move the milestone before I forget. |
# Conflicts: # WooCommerce/Classes/ViewRelated/Dashboard/Free Trial/StorePlanBannerPresenter.swift # WooCommerce/Classes/ViewRelated/Upgrades/CurrentPlanDetailsView.swift # WooCommerce/Classes/ViewRelated/Upgrades/UpgradeWaitingView.swift # WooCommerce/WooCommerce.xcodeproj/project.pbxproj
|
Additionally there are 2 subscription groups (9 subscriptions total, production and test) that we can remove from ASC, as well as the 2 .storekit config files from the Woo target. I'll clean these after we've wrapped this one up. |
|
I've not forgotten about this, will get to it in the next day or two |
I don't think you can ever remove approved subscriptions from ASC, just mark them as not available for sale, which should be done already |
joshheald
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done. Always good to see a summer's worth of work deleted 😂
There may be some stuff you can remove from PaymentAction/Store... worth checking with Kiwi whether we actually do the check for whether it's a free trial or not and show publishing stuff as part of the onboarding. It smells like Woo Express code to me. Another PR might be best though
# Conflicts: # WooCommerce/WooCommerce.xcodeproj/project.pbxproj
|
Thanks for the review!
😂
Yes, there are still bits of IAP/WooExpress/Domains here and there that wasn't coupled with the rest of IAP, I'll clean up the other bits on a separate PR so this one doesn't get even bigger. |
|
Version |
Does not compile after modularization due to error: Build input files cannot be found. 4 files still appear referenced in the woo target build phase despite being deleted, so I’ve removed them manually.

Closes WOOMOB-1290
Description
This PR gets rid of the In-App Purchases implementation, as well as partially of the usage of Plan/Domain upgrades through onboarding flow, since some bits of this flow was also using IAP internals. Confirmed with Kiwi that is no longer needed here: p1757921125873789-slack-C03L1NF1EA3, but still needs some extra task to remove the remains that are not coupled, which can be done separately.
There has been no IAP purchases (merchants were not able to purchase since the feature was disabled) or renewals (existing purchases could still be renewed via the app) since April 5th, 2024, so we should be good to remove it. Android counterpart was done in January: pdfdoF-6aJ-p2
Testing information