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

Don't take the item when the transaction failed #14

Merged
merged 4 commits into from
Sep 18, 2016
Merged

Don't take the item when the transaction failed #14

merged 4 commits into from
Sep 18, 2016

Conversation

Luca0208
Copy link
Contributor

This PR can fix issue #13

@Luca0208
Copy link
Contributor Author

don't merge this yet, my IDE has done a few strange things.

@Luca0208
Copy link
Contributor Author

Ok I fixed it in the browser really quick

@Luca0208
Copy link
Contributor Author

Luca0208 commented Sep 16, 2016

My IDE will drive me crazy at some point. But at least it decided that now would be the right moment to push working code. 🎉

@hsyyid hsyyid merged commit 7d21641 into hsyyid:master Sep 18, 2016
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