Skip to content

[fix] Loader state for all 3 types of cards #28

@Hamdrive

Description

@Hamdrive

Noticed that the current implementation of loader state for all 3 cards (Product, Wishlist and Cart) are using the same shared context state,

  1. This firstly might go against the concept of a global shared state with context
  2. This results in one API request triggering loader for all cards on the page

This issue is more apparent and visible on the cart page, when user tries to either remove the product or move to wishlist

Suggestions:

  1. Have separate state for all 3 cards, will result in repetitive states but each will be unique to only that card, ensuring each component is triggered does not affect other children

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions