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

Update to latest PaypalSdk #2

Open
tom-mayer opened this issue Mar 23, 2015 · 4 comments
Open

Update to latest PaypalSdk #2

tom-mayer opened this issue Mar 23, 2015 · 4 comments

Comments

@tom-mayer
Copy link

Hi,

I tried updating your library project to the lates PaypalSDK, since the early 2.x Versions have some rather serious bugs if the native Paypal app is installed on the device.

I updated the JAR file and added the extra native libraries to your SDK project and tried compiling it. However I get the following exception.

screen shot 2015-03-23 at 10 13 16

I also started working on updating our own wrapper here:
https://github.com/tom-mayer/Mono.Droid.Paypal
this however doesn't build since an interface on a generated class could not be resolved.

I don't really know much about Java binding since up until now I have only bound ObjectiveC classes. Would you be interested in merging the projects or in working together on upgrading?

Cheers
Tom

@raechten
Copy link
Owner

Hi Tom,

Merging sounds like a good idea, since there is no point in two similar projects. I have not had a chance to update in a long time since I am doing other work now. Like you, I am by no means an expert in Java bindings. I managed to get the binding for that particular version working by reading the documentation and trial-and-error, but it was a time-consuming project.

As I said, I have not been able to work on it for a while now, so if you want to take ownership, let me know and we’ll get it going. I will try to get some time to find out why the new version doesn’t like the binding. They seem to be moving stuff around a lot in the PayPal SDK and they seem to have the code obfuscated.

Kind regards,
Robert

Op 23 mrt. 2015, om 02:18 heeft Tom Mayer notifications@github.com het volgende geschreven:

Hi,

I tried updating your library project to the lates PaypalSDK, since the early 2.x Versions have some rather serious bugs if the native Paypal app is installed on the device.

I updated the JAR file and added the extra native libraries to your SDK project and tried compiling it. However I get the following exception.

https://cloud.githubusercontent.com/assets/2562379/6776970/64b2efce-d145-11e4-82e2-48455c9dd930.png
I also started working on updating our own wrapper here:
https://github.com/tom-mayer/Mono.Droid.Paypal https://github.com/tom-mayer/Mono.Droid.Paypal
this however doesn't build since an interface on a generated class could not be resolved.

I don't really know much about Java binding since up until now I have only bound ObjectiveC classes. Would you be interested in merging the projects or in working together on upgrading?

Cheers
Tom


Reply to this email directly or view it on GitHub #2.

@tom-mayer
Copy link
Author

Hi Robert,

I manged to get the bindings working for the 2.8.8 version. I took the repo offline since i need to discuss something with the client first. I will definitely put it online again later this week. I'll contact you here when its up again, then we can discuss how we do it, I will probably just make a pull request for you, or you can add me to the collaborators.

Cheers and thanks for your quick reply!
Tom

@raechten
Copy link
Owner

Hi Tom,

Excellent! I’ll be waiting for your email or if you want, I can make you a collaborator now. Providing I know how to do this… :-P

Cheers,
Robert

Op 23 mrt. 2015, om 08:45 heeft Tom Mayer notifications@github.com het volgende geschreven:

Hi Robert,

I manged to get the bindings working for the 2.8.8 version. I took the repo offline since i need to discuss something with the client first. I will definitely put it online again later this week. I'll contact you here when its up again, then we can discuss how we do it, I will probably just make a pull request for you, or you can add me to the collaborators.

Cheers and thanks for your quick reply!
Tom


Reply to this email directly or view it on GitHub #2 (comment).

@d87manish
Copy link

Hi Robert,

I am looking forward for update in sdk against this issue:
paypal/PayPal-Android-SDK#117

Please have a look and let me know. I am having the similar issue.

Thanks
Manish

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

No branches or pull requests

3 participants