-
Notifications
You must be signed in to change notification settings - Fork 21
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 the billing setup page from the Ads setup flow #2536
Comments
@eason9487 could you please review this one as well. I'd like for this to be able to be worked in parallel with #2535, if possible, so we may need some follow-up once these are merged into the target feature branch. |
Since it might be realized at the time of development that they need to modify more of the same files, I'm inclined to suggest doing them sequentially. But before considering the possibility of working in parallel, maybe the doubt in #2534 (comment) needs to be clarified first as this ticket has the same situation. |
Agreed. I'll get that verified before moving on. |
I've clarified with @fblascogarma that we'll remove this step along with the first step in #2534 and consolidate the need to set up billing to the unified campaign setup screen in #2535. |
E2E for this task is fixed. This is good for WooCR but currently blocked by a strange error in GH action. https://github.com/woocommerce/google-listings-and-ads/actions/runs/11057375294/job/30725249745 |
@eason9487 looks like this error has now been fixed. |
Part of #2460
When a merchant is setting up a new ads campaign from the plugin dashboard, they enter the "Set up paid campaign" flow, which includes individual steps for connecting a Google Ads account, configuring a campaign, and completing their billing setup.
In #2535, we are replacing the second step with the same component used during onboarding, which includes the ability to set up billing if needed. That makes the "Setup up billing" step redundant and can be removed.
Acceptance Criteria
AdsStepper
.Implementation Brief
The PR should get merged into the feature/2460-simplify-paid-ads-setup branch.
The
AdsStepper
component for the "Set up paid campaign" flow is injs/src/setup-ads/ads-stepper/index.js
. The billing step can be removed from the array passed to thesteps
prop here.The
handleSubmit
prop that is passed via formProps to theSetupBilling
component will need to be moved to the component responsible for rendering the ads campaign setup (e.g.AdsCampaign
) so that step can complete the form when finished.For testing, billing status can be mocked using the following filter in an MU plugin:
Test Coverage
The text was updated successfully, but these errors were encountered: