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

Added "Add / update" comment. Update was missing. #7541

Merged
merged 2 commits into from
Mar 17, 2017
Merged

Added "Add / update" comment. Update was missing. #7541

merged 2 commits into from
Mar 17, 2017

Conversation

gastondisacco
Copy link
Contributor

@gastondisacco gastondisacco commented Nov 23, 2016

Should have the same description as in CartItemRepositoryInterface.php.
Currently only says add, but the API works for add and update too.

Should have the same description as in CartItemRepositoryInterface.php.
Currently only says add, but the API works for add and update too.
@gastondisacco
Copy link
Contributor Author

"Added "Add / update" comment. Update was missing." was the original title of the post, changed it to notice travis error.

@gastondisacco gastondisacco changed the title Added "Add / update" comment. Update was missing. continuous-integration/travis is failing! Nov 24, 2016
@dverkade
Copy link
Member

Hi,

I have head the same problem. The build is failling on calling a URL for fetching the exchange rates. I have rebased my commit a couple of times and finally it passed. You might want to try this as well.

@gastondisacco
Copy link
Contributor Author

Hi @dverkade, thanks for your comment! Will try again so.

@gastondisacco gastondisacco changed the title continuous-integration/travis is failing! Added "Add / update" comment. Update was missing. Nov 25, 2016
@davidalger davidalger self-assigned this Mar 16, 2017
@davidalger davidalger added this to the March 2017 milestone Mar 16, 2017
@davidalger davidalger self-requested a review March 16, 2017 02:48
@okorshenko okorshenko self-assigned this Mar 16, 2017
@magento-team magento-team merged commit 5d5d9d9 into magento:develop Mar 17, 2017
@gastondisacco gastondisacco deleted the patch-1 branch March 17, 2017 01:42
@okorshenko
Copy link
Contributor

@gastondisacco thank you for your contribution!
@davidalger thank you for processing this PR

magento-devops-reposync-svc pushed a commit that referenced this pull request Apr 11, 2022
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.

5 participants