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

Add manual purchase finish #121

Closed

Conversation

lunarraid
Copy link

No description provided.

@lunarraid lunarraid closed this Sep 22, 2017
@lazyhoneyant
Copy link

@lunarraid Why do you close this PR? I use your PR well. Is there any problem on this PR?

@lunarraid
Copy link
Author

This PR was made in error. I meant to pull the change into the master branch of my fork only, as this change breaks compatibility with the main repo.

@lazyhoneyant
Copy link

@lunarraid I understand. I've used this PR to my fork repo. Thanks for sharing your useful code.

@Jacse
Copy link

Jacse commented Nov 6, 2017

@lunarraid this is extremely useful and necessary for the library. Do you think you could be convinced to merge your functionality into the lib?

@lunarraid
Copy link
Author

I have no problem with re-opening this if the maintainer is cool with the breaking changes.

@chirag04
Copy link
Owner

chirag04 commented Nov 6, 2017

I'm fine with making a breaking change. Refer the thread here: #124 (comment)

@lunarraid
Copy link
Author

#135 has been opened.

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.

4 participants