Don't deploy specific payment provider settings

[mattbrailsford]

This was raised in the Vendr Deploy repo but probably counts here too, but some payment provider settings might not want to be deployed so we have implemented an app setting to ignore specific settings. Whatever we do in that package, we are probably going to need to replicate here too

vendrhub/vendr-deploy#14

[KevinJump]

Hi,

easing back into uSync.Vendr with this one. :)

So just to confirm what i am looking at ?

In a "PaymentMethod" we have a provider settings section ?

e.g for the 'Zero Value' payment methiod.

  <ProviderSettings>
    <Setting>
      <Key>ContinueUrl</Key>
      <Value>/checkout/order-confirmation/</Value>
    </Setting>
  </ProviderSettings>

presumably for more complex payment systems this is fancier and has more properties ?

and we want to ignore/not serialize/deserialize certain ones?

[mattbrailsford]

Yea, so those settings are always key/value, but in the app settings there should be something to say "ignore this setting entirely" and just don't include it.

For example, here's what we now support in Vendr.Deploy

{
   ...
   "Vendr.Deploy": {
        "PaymentMethods": {
            "IgnoreSettings": [ "testMode", "sanboxMode", "liveApiKey" ]
        }
    }
}

Which means any settings with the key testMode, sandboxMode or liveApiKey will get ignored and won't be transfered, so ideally I guess we would want to support

{
   ...
   "Vendr.uSync": {
        "PaymentMethods": {
            "IgnoreSettings": [ "testMode", "sanboxMode", "liveApiKey" ]
        }
    }
}

[KevinJump]

there is already have a way of passing settings to the serializers

so the "simple" fix is to use that, but the settings will be buried in uSync then.

 "uSync": {
    "Publisher": {
      "Handlers": {
        "vendrPaymentMethodHandler": {
          "settings": {
            "IgnoreSettings": [ "testMode", "sanboxMode", "liveApiKey" ]
          }
        }
      }
  }
}

as much as that obviously pleases me, i think we can make a vendr.usync settings section for this one 😄

[KevinJump]

Hi this is already done on serialization 😄
5ed72ec

I suppose for completeness, we could do in on import too, just incase its not configured on the exporting server.

[mattbrailsford]

Good point, I don't actually think we do that in Vendr.Deploy so I should probably implement that too 😁