Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

Fix error#109 #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix error#109 #110

wants to merge 1 commit into from

Conversation

ryangrunest
Copy link

Changes start in line 126 in the addVariantToCart() function.

@ryangrunest
Copy link
Author

Fix for #109

@swalkinshaw
Copy link
Collaborator

Thanks @ryangrunest. Is it possible to just get the necessary changes in here? It's really hard to tell what the actual fix is vs other changes.

@enkelmedia
Copy link

The diff is impossible to read, what did you change to get this to work?

@enkelmedia
Copy link

enkelmedia commented Jan 21, 2021

Ahh, L128, just adding const client = this.props.client;

From:

addVariantToCart(variantId, quantity){
    ...
}

To:

addVariantToCart(variantId, quantity){
    const client = this.props.client;
    ...
}

@enkelmedia
Copy link

@swalkinshaw can you just fix this or do you want a new PR?

@swalkinshaw
Copy link
Collaborator

I can't push commits to his branch, so if you want to do a new PR then that would be helpful 👍

@enkelmedia
Copy link

PR here: #130

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

Successfully merging this pull request may close these issues.

None yet

3 participants