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

Enhance Payment Page Functionality #695

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Rahul-AkaVector
Copy link

@Rahul-AkaVector Rahul-AkaVector commented Oct 20, 2024

Pull Request: Enhance Payment Page Functionality

Description

This pull request builds upon the previous PR titled "Redesigning Payment Page" (#567), which was merged 4 days ago. The goal of this enhancement is to address issues related to the dynamic display of cart items and total price on the payment page, as highlighted in the previous review by @dohinafs .

Changes Made

In response to the feedback received, we have made significant improvements to the functionality of the payment page:

  1. Dynamic Order Summary:

    • The functionality now correctly calculates and displays the total number of items in the cart and their total price.
    • Each item is displayed individually, even if there are multiple quantities, ensuring a clear and accurate representation of the order.
  2. Updated Cart Management:

    • The logic for adding and removing items from the cart has been refined to ensure accurate tracking of quantities.
    • The displayCart function has been enhanced to dynamically show each item added to the cart along with its quantity and price, addressing the issue where the page displayed static values.
  3. User Interface Improvements:

    • Improved layout and visibility adjustments for better readability.
    • Ensured that the user experience is smooth and intuitive, particularly when handling multiple items.

Related Issue

Closes 📝: Redesigning Payment Details Page #495

Checklist

  • My code follows the project's coding style.
  • I have tested my changes locally.
  • I have updated the documentation (if necessary).
  • The changes I made do not break existing functionality.

Copy link

netlify bot commented Oct 20, 2024

Deploy Preview for arcticdelights ready!

Name Link
🔨 Latest commit f4d0750
🔍 Latest deploy log https://app.netlify.com/sites/arcticdelights/deploys/6714bb39929edb0008ce7403
😎 Deploy Preview https://deploy-preview-695--arcticdelights.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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants