stripe_commerce icon indicating copy to clipboard operation
stripe_commerce copied to clipboard

Added Variant SKU many places.

Open djdarkbeat opened this issue 12 years ago • 7 comments

Added some variant SKU in many places to ease tracking of variants in cart and orders.

djdarkbeat avatar Nov 18 '13 02:11 djdarkbeat

I am using ROR-e framework to implement shopping cart.I found critical issue# If user login with User ID [email protected] on Chrome browser and add 3 items to shopping cart for e.g. 3 items. Now if user login with same User ID= [email protected] on Firefox or IE or Safari browser and click on the shopping cart, It shows 0 items.It's suppose to show 3 items, since it's a same User ID. So ROR-e framework is not storing items in shopping cart based on User,instead it is storing in session. Please let me know if shopping cart is session based or user based or both. Regards, Pravin 917-514-0765 screen shot 2014-04-10 at 4 23 39 pm screen shot 2014-04-10 at 4 25 09 pm

pkamble avatar Apr 10 '14 20:04 pkamble

It stores both depending on if you are logged in or not

I'll test it out but it should work On Thursday, April 10, 2014 1:30 PM, mkrsqr [email protected] wrote:

I am using ROR-e framework to implement shopping cart.I found critical issue# If user login with User ID [email protected] on Chrome browser and add 3 items to shopping cart for e.g. 3 items. Now if user login with same User ID= [email protected] on Firefox or IE or Safari browser and click on the shopping cart, It shows 0 items.It's suppose to show 3 items, since it's a same User ID. So ROR-e framework is not storing items in shopping cart based on User,instead it is storing in session. Please let me know if shopping cart is session based or user based or both. Regards, Pravin 917-514-0765

— Reply to this email directly or view it on GitHub.

drhenner avatar Apr 10 '14 22:04 drhenner

Thanks David for response.I tested on Tonx.org, implementation of ROR-E and it's working. But it's not working on DEMO as well as on our implementation as well. In my test case, I logged in first and then add items to cart. Please let me know if i am making any mistake.

Also, we are planning got use your Stipe_Commerce and would like to make payment $250. Please let me know how can we do it.

On Thu, Apr 10, 2014 at 6:21 PM, D. R. Henner [email protected]:

It stores both depending on if you are logged in or not

I'll test it out but it should work On Thursday, April 10, 2014 1:30 PM, mkrsqr [email protected] wrote:

I am using ROR-e framework to implement shopping cart.I found critical issue# If user login with User ID [email protected] on Chrome browser and add 3 items to shopping cart for e.g. 3 items. Now if user login with same User ID= [email protected] on Firefox or IE or Safari browser and click on the shopping cart, It shows 0 items.It's suppose to show 3 items, since it's a same User ID. So ROR-e framework is not storing items in shopping cart based on User,instead it is storing in session. Please let me know if shopping cart is session based or user based or both. Regards, Pravin 917-514-0765

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/drhenner/stripe_commerce/pull/3#issuecomment-40150076 .

pkamble avatar Apr 10 '14 22:04 pkamble

Pravin,

I'll try to see what I can do to get this working on the weekend. My arm is getting out of a cast this weekend. (long story but broke it in a bike accident)

As for a payment, Lets chat outside of github. drhenner [at] ror-e.com

Dave

drhenner avatar Apr 10 '14 23:04 drhenner

@mkrsqr the merging carts logic should work now. I have not manually tested this yet but I added test and it appears to work as expected. If you have any issues ping me. I have to run to brunch!

drhenner avatar Apr 13 '14 18:04 drhenner

David,

I tested it manually couple of time and i can reproduce it on demo version you deployed. I attached screenshot on github. It's strange, it works on www.tonx.org which is implementation ROR-e commerce.

Regards, Pravin

On Sun, Apr 13, 2014 at 2:46 PM, D. R. Henner [email protected]:

@mkrsqr https://github.com/mkrsqr you merging carts logic should work now. I have not manually tested this yet but I added test and it appears to work as expected. If you have any issues ping me. I have to run to brunch!

Reply to this email directly or view it on GitHubhttps://github.com/drhenner/stripe_commerce/pull/3#issuecomment-40315806 .

pkamble avatar Apr 14 '14 01:04 pkamble

I didn't deploy this to the demo. I'll get to that later if you want but first I need to do my taxes... :frowning:

drhenner avatar Apr 14 '14 03:04 drhenner