Merge pull request #418 from recurly/paypal-logo

Adds support for logo assignment in PayPal flow
recurly · Jan 16, 2018 · 617e326 · 617e326
2 parents 0bd4242 + 4201b74
commit 617e326
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/lib/recurly/paypal/strategy/direct.js b/lib/recurly/paypal/strategy/direct.js
@@ -12,11 +12,16 @@ export class DirectStrategy extends PayPalStrategy {
     this.emit('ready');
   }
 
-  start () {
-    // TODO: More parameters here
+  get payload () {
     let payload = { description: this.config.display.displayName };
     if (this.config.display.amount) payload.amount = this.config.display.amount;
+    if (this.config.display.logoImageUrl) payload.logoImageUrl = this.config.display.logoImageUrl;
+    if (this.config.display.headerImageUrl) payload.headerImageUrl = this.config.display.headerImageUrl;
+    return payload;
+  }
 
+  start () {
+    const { payload } = this;
     const frame = this.recurly.Frame({ path: '/paypal/start', payload });
     frame.once('error', cause => this.error('paypal-tokenize-error', { cause }));
     frame.once('done', token => this.emit('token', token));

diff --git a/lib/recurly/paypal/strategy/index.js b/lib/recurly/paypal/strategy/index.js
@@ -15,7 +15,11 @@ const DISPLAY_OPTIONS = [
   'shippingAddressOverride',
   'shippingAddressEditable',
   'billingAgreementDescription',
-  'landingPageType'
+  'landingPageType',
+
+  // Not compatible with Braintree options due to lack of support
+  'logoImageUrl',
+  'headerImageUrl'
 ];
 const DEFAULTS = {
   display: {