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

calendar event deletion #16

Merged
merged 2 commits into from Oct 18, 2014

Conversation

Projects
None yet
2 participants
@ljanvier
Contributor

ljanvier commented Oct 7, 2014

No description provided.

comewalk added a commit that referenced this pull request Oct 18, 2014

@comewalk comewalk merged commit 47dcfa8 into comewalk:master Oct 18, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@comewalk

This comment has been minimized.

Show comment
Hide comment
@comewalk

comewalk Oct 18, 2014

Owner

@ljanvier LGTM 👍 Thanks!

Owner

comewalk commented Oct 18, 2014

@ljanvier LGTM 👍 Thanks!

@comewalk comewalk referenced this pull request Oct 18, 2014

Open

Add response method #17

@comewalk

This comment has been minimized.

Show comment
Hide comment
@comewalk

comewalk Oct 18, 2014

Owner

@ljanvier sorry, I will revert this PR at #17. Because execute method should return contents of response. In case of status code is 204, it should see status code, I think. I will be fix this at #17.

Owner

comewalk commented Oct 18, 2014

@ljanvier sorry, I will revert this PR at #17. Because execute method should return contents of response. In case of status code is 204, it should see status code, I think. I will be fix this at #17.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment