Skip to content

RPP / Failures / Cleanup #7426

Open
Open
@RadoslavGeorgiev

Description

@RadoslavGeorgiev

Description

Make sure all cleanup tasks are executed upon failure within the new payment process.

It seems like the gradual migration to split-UPE with deferred intent is leaving less and less to clean up.

Presently, the only thing that I can find in place regarding cleanup is the following. However, please check the existing payment process, and make sure that no other clean-up tasks need to be performed during cleanup, or if there are, implement them.

if ( WC_Payments_Features::is_upe_legacy_enabled() ) {
UPE_Payment_Gateway::remove_upe_payment_intent_from_session();
}

Considering that we will never enable the new payment process alongside legacy UPE, that will not be needed.

Acceptance criteria

All cleanup tasks, if any, and except the example above, are executed.

Testing instructions

To be defined, based on the findings from above.

Additional context

Epic: #6673
Project: paJDYF-9hL-p2

Metadata

Metadata

Assignees

Labels

category: coreWC Payments core related issues, where it’s obvious.category: engineeringFor product engineering, architecture work, tech debt and so on.category: projectsFor any issues which are part of any project, including bugs, enhancements, etc.component: checkoutIssues related to Checkoutfocus: checkout paymentsstatus: blockedThe issue is blocked from progressing, waiting for another piece of work to be done.status: on holdThe issue is currently not prioritized.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions