Required changes for a unified Ti.PayPal module (Parity)

[hansemannn]

Hey @AppWerft,

let's write up a list of parity-changes required to move the iOS and Android version together:

• Rename initPayment to initialize
• Remove the ability to store keys in the tiapp.xml (security reasons, the keys are not encrypted I think) - No blocker
• Require the configuration key to be an instance of the Configuration proxy (no object, to keep it more structured and be able to hook into possible events that will be exposed in the future)
• Require the items key to be an array of the PaymentItem proxy instances (see above reasons). I noticed there is no PaymentItem proxy so far, so if possible, create that one for parity
• Ensure all events from iOS also exist on Android or list the ones that are not covered. Some examples of important events: paymentDidCancel , paymentWillComplete and paymentDidComplete (For example, I don't see paymentWillComplete in the Android docs so far)

Thank you so much for the efforts! It will be amazing to have a centralized and up-to-date PayPal module for the developers again!

[AppWerft]

I will extend with compatibilty to old version. Am 24.05.17 um 11:49 schrieb Hans Knöchel:

…

Hey @AppWerft <https://github.com/appwerft>, let's write up a list of parity-changes required to move the iOS and Android version together: * Rename |initPayment| to |initialize| * Remove the ability to store keys in the tiapp.xml (security reasons, the keys are not encrypted I think) - No blocker * Require the |configuration| key to be an instance of the |Configuration| proxy (no object, to keep it more structured and be able to hook into possible events that will be exposed in the future) * Require the |items| key to be an array of the |PaymentItem| proxy instances (see above reasons). I noticed there is no |PaymentItem| proxy so far, so if possible, create that one for parity * Ensure all events from iOS also exist on Android or list the ones that are not covered. Some examples of important events: |paymentDidCancel|, |paymentWillComplete| and |paymentDidComplete| (For example, I don't see |paymentWillComplete| in the Android docs so far) Thank you *so much* for the efforts! It will be amazing to have a centralized and up-to-date PayPal module for the developers again! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#8>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC24DZNZLQnc-6XWScVlg_FQvl5KR_7Vks5r8_06gaJpZM4Nk3Jg>.

-- *Dipl.-Ing. Rainer Schleevoigt* Certified TYPO3 Integrator Certified Titanium App Developer Novalisweg 10 22303 Hamburg ℡ 040 60812460 |Ust-ID: DE239491976|||||||