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

Walmart V2 to V3 price update changes #50

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

jpnathannew
Copy link

Dear team,

Hope you are doing well. I have made some changes in src/price.php and src/description/price.php for price update version upgrade V2 to V3. Actually I tried with V3 price update. But that one is not working. But Bulk price is working fine with out any issues.

Thanks,
Nathan

Walmart V3 price update needs WM_CONSUMER.CHANNEL.TYPE for authentication. So added WM_CONSUMER.CHANNEL.TYPE into config variable.
V3 price update, we are using V3 Bulk price update. So in this file I updated apiVersion V2 to V3.
@coveralls
Copy link

Coverage Status

Coverage decreased (-4.3%) to 91.928% when pulling 0500cee on jpnathannew:develop into 8344dc8 on fillup:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-4.3%) to 91.928% when pulling 0500cee on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage decreased (-4.3%) to 91.928% when pulling 0500cee on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e965af1 on jpnathannew:develop into 8344dc8 on fillup:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e965af1 on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e965af1 on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e6a0d6f on jpnathannew:develop into 8344dc8 on fillup:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e6a0d6f on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e6a0d6f on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

coveralls commented Apr 17, 2017

Coverage Status

Coverage increased (+0.1%) to 96.321% when pulling e6a0d6f on jpnathannew:develop into 8344dc8 on fillup:develop.

Single Product Price update API not working on V3. So testUpdate function removed from test case
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.305% when pulling f45dab0 on jpnathannew:develop into 8344dc8 on fillup:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.305% when pulling f45dab0 on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 96.305% when pulling f45dab0 on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 96.374% when pulling 1153abc on jpnathannew:develop into 8344dc8 on fillup:develop.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 96.374% when pulling 1153abc on jpnathannew:develop into 8344dc8 on fillup:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 96.374% when pulling 1153abc on jpnathannew:develop into 8344dc8 on fillup:develop.

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

Successfully merging this pull request may close these issues.

2 participants