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

feat(cart): carts pages #990

Merged

Conversation

Sinucid
Copy link
Contributor

@Sinucid Sinucid commented Dec 12, 2023

  • carts page (/my-account/carts) was added to the storefront preset
  • TokenResolve's resolveToken method is accepting options object now instead string token ({ token: string, contextElement?: HTMLElement | LitElement}), with token and optional context element that can be used as context provider
  • Cart token resolver's SUMMARY and EMPTY are able to resolve values within context now
  • Cart and Order totals resolvers are also resolving the totals depends on the context
  • CompositionComponentsController is providing host element now as a contextElement for dynamic visibility rules, to be able to resolve resources within context

closes: HRZ-90718, HRZ-90086

@Sinucid Sinucid requested a review from a team as a code owner December 12, 2023 14:42
Copy link

netlify bot commented Dec 12, 2023

Deploy Preview for dev-oryx-components ready!

Name Link
🔨 Latest commit 323b4d9
🔍 Latest deploy log https://app.netlify.com/sites/dev-oryx-components/deploys/6578715a9f257b00087f9b26
😎 Deploy Preview https://deploy-preview-990--dev-oryx-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 12, 2023

Deploy Preview for dev-fulfillment canceled.

Name Link
🔨 Latest commit 323b4d9
🔍 Latest deploy log https://app.netlify.com/sites/dev-fulfillment/deploys/6578715a6490240008aa9443

Copy link

netlify bot commented Dec 12, 2023

Deploy Preview for dev-composable-storefront ready!

Name Link
🔨 Latest commit 323b4d9
🔍 Latest deploy log https://app.netlify.com/sites/dev-composable-storefront/deploys/6578715a486ed80008e777ab
😎 Deploy Preview https://deploy-preview-990--dev-composable-storefront.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
3 paths audited
Performance: 32 (🟢 up 4 from production)
Accessibility: 90 (no change from production)
Best Practices: 97 (no change from production)
SEO: 87 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@Sinucid Sinucid changed the base branch from development to task/HRZ-90000-multi-cart-list December 12, 2023 14:42
Copy link

cypress bot commented Dec 12, 2023

1 flaky test on run #11816 ↗︎

0 57 57 0 Flakiness 1

Details:

hrz-90086: cr fixes
Project: Composable Storefront Commit: 33034b1e9b
Status: Passed Duration: 07:09 💡
Started: Dec 14, 2023 8:55 AM Ended: Dec 14, 2023 9:02 AM
Flakiness  src/integration/checkout.cy.ts • 1 flaky test • sf-regression

View Output Video

Test Artifacts
Checkout suite > for authenticated user: > should create a new order successfully Test Replay Screenshots Video

Review all test suite changes for PR #990 ↗︎

@Sinucid Sinucid changed the title feat(cart): carts and cart details pages feat(cart): carts pages Dec 13, 2023
Copy link

cypress bot commented Dec 13, 2023

Passing run #2698 ↗︎

0 5 0 0 Flakiness 0

Details:

hrz-90086: cr fixes
Project: Fulfillment App Commit: 33034b1e9b
Status: Passed Duration: 07:19 💡
Started: Dec 14, 2023 9:22 AM Ended: Dec 14, 2023 9:30 AM

Review all test suite changes for PR #990 ↗︎

@Sinucid Sinucid requested a review from dunqan December 14, 2023 08:54
@Sinucid Sinucid merged commit 7880776 into task/HRZ-90000-multi-cart-list Dec 14, 2023
23 checks passed
@Sinucid Sinucid deleted the feature/hrz-90086-carts-routes branch December 14, 2023 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants