-
Notifications
You must be signed in to change notification settings - Fork 219
Conversation
The release ZIP for this PR is accessible via:
Script Dependencies ReportThe
This comment was automatically generated by the TypeScript Errors Report
assets/js/blocks/order-confirmation/additional-information/index.tsx
assets/js/blocks/order-confirmation/billing-wrapper/index.tsx assets/js/blocks/order-confirmation/downloads-wrapper/index.tsx assets/js/blocks/order-confirmation/shipping-wrapper/index.tsx assets/js/blocks/order-confirmation/status/index.tsx assets/js/blocks/order-confirmation/summary/index.tsx assets/js/blocks/order-confirmation/totals-wrapper/index.tsx assets/js/blocks/order-confirmation/totals/index.tsx |
Size Change: +24.8 kB (+2%) Total Size: 1.4 MB
ℹ️ View Unchanged
|
80b2f19
to
49893c6
Compare
79d5fc8
to
0ac6f1f
Compare
be361f6
to
8b638b6
Compare
This PR has been marked as If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label. |
This PR has been marked as If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label. |
4d7c0ce
to
47e53ee
Compare
f087f82
to
de8079f
Compare
f1a1a12
to
7eaa78d
Compare
e2e tests are still needed, and a decision on #10050, but this feature branch is close to feature complete so I think we're ok to start the final review and approval process @tarhi-saad. Marking as ready to review. |
* Add block which contains hooks * Use skeleton for placeholder instead of illustration * Remove duplicate methods
* Add tests for conditional blocks * Move setup into test
* Add editor util functions * Update editor template E2E test * Add the "exact" property for consistency * Skip test Can't get the element in the page. More investigation needed! Skipping for now. * Fix "transformIntoBlocks" logic * Add tests for logged in user * Fix "beforeAll" config * Confirm downloads section is visible when logged in * Create "verifyOrderConfirmationDetails" util function * Add logged in test case * Add Guest user test case * Fix editor e2e testing * Apply a proper teardown * Fix failing tests after logout * Ensure we are logged in before visiting the editor * Ensure to have shipping selected * Wait for changes to be saved on the editor * Ensure shipping options is selected * Remove comment * Ensure we are logged in before going to the admin page * Mark the Order Confirmation as a side effect test
090713f
to
5221768
Compare
Login out causes other tests to fail. We will implement these case when the multiple sign in roles are introduced in the codebase.
What
This PR introduces new blocks for the individual parts of the order confirmation page. This includes blocks to display:
This replaces the legacy confirmation block which can be found under Appearance > Editor > Templates > Order Confirmation. The switch between legacy and new blocks is a manual process that involves editing the template and clicking the "transform into blocks" button.
Fixes #10021
Why
This change allows merchants to control the appearance of the order confirmation page. It also modernises how the order confirmation page is rendered.
Testing Instructions
Perform the following tests as a logged in admin user unless otherwise specified.
Downloadable Products
Unauthenticated Views
checkout/order-received/1772/?key=wc_order_GAE4XxQ4k7E77
would becomecheckout/order-received/1772
Developer tests
Install a 3rd party extension which adds something to the order confirmation page. Confirm that the extension renders it's content on the new order confirmation page. You can find examples on WooComerce.com. One I tried was "Share your purchase" which appends social sharing to the confirmation page.
Screenshots or screencast
WooCommerce Visibility
Required:
Checklist
Required:
[type]
label or a[skip-changelog]
label.Conditional:
[skip-changelog]
label is not present).Changelog