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 logPurchase support #16

Closed
wants to merge 1 commit into from
Closed

Conversation

Protossoario
Copy link
Contributor

Allows users to log purchases through the Facebook SDK specifying the currency

@ashcoding ashcoding closed this May 27, 2015
@ashcoding ashcoding reopened this May 27, 2015
@ashcoding
Copy link
Contributor

@Protossoario Thanks for the PR. Before we could review this, could you sign the CLA?
https://cla.appcelerator.com/

@ashcoding
Copy link
Contributor

Also, could you undo/remove the file change of android/build.properties.example → android/build.properties. Thank you!

@Protossoario Protossoario force-pushed the master branch 2 times, most recently from e3637c9 to 713ad5d Compare May 27, 2015 16:31
@Protossoario
Copy link
Contributor Author

I successfully removed the change of build.properties.example and signed the CLA!

@ashcoding ashcoding closed this May 28, 2015
@ashcoding ashcoding reopened this May 28, 2015
@ashcoding
Copy link
Contributor

@Protossoario sorry to bother you again. The auto check didn't show your github account as signed. Did you sign it here https://cla.appcelerator.com/ or another location?

Also, could you rebase/merge all the commits into one? Seems like some other commit history came along with it after you removed that file. If you could do this on this branch http://makandracards.com/makandra/527-squash-several-git-commits-into-a-single-commit (Easy mode: Reset your feature branch to the master state) and then force push the updated branch, that would be awesome!

Sorry for the request! And thanks again!

@Protossoario
Copy link
Contributor Author

Thanks for the link! It was really easy to do the reset with the instructions on that page and the pull request should be a lot cleaner now.

As for the CLA, I can't seem to sign it. I try clicking the "Sign in with GitHub" button, but it keeps redirecting me to the same page over and over.

@ashcoding
Copy link
Contributor

@Protossoario thanks! With regards to the CLA, could you try again? Perhaps it's something to do with your Github settings...

@ashcoding ashcoding closed this Jun 8, 2015
@ashcoding ashcoding reopened this Jun 8, 2015
@Protossoario
Copy link
Contributor Author

Finally! It seems there was an issue with Chrome which was preventing me from signing the CLA. I tried with Safari and it finally worked just fine.

@hansemannn
Copy link
Contributor

@ashcoding Can we tackle this one? We will also need docs updates here, would be great if @Protossoario could provide them! :-)

@ashcoding
Copy link
Contributor

@hansemannn Yup. This is okay.

@hansemannn
Copy link
Contributor

hansemannn commented Feb 24, 2016

@Protossoario Any update here? Has been some more months now. As the code currently also has merge problems, please also rebase your PR to ensure the continuous integration. Also think of the docs. Thank you!

@hansemannn
Copy link
Contributor

hansemannn commented Apr 17, 2016

Finally resolved all merge conflicts, added the iOS implementation, docs, JIRA-ticket and example. Closing in favor of: #49

@hansemannn hansemannn closed this Apr 17, 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.

None yet

3 participants