Fix type (#446)

Author: markulrich

packages/donate-button-v4/src/components/widget/components/PaymentProcess/PaymentMethodSelect/index.tsx

@@ -45,8 +45,11 @@ const usePaymentMethods = () => {
 		() =>
 			methods
 				.filter((method) => {
+					// There is a bug where instead of being a proper array of disabled methods,
+					// it comes as an array of length 1 with a comma-separated string inside
 					if (
-						nonprofit?.metadata?.disabledPaymentFlowOptions?.includes(method)
+						nonprofit?.metadata?.disabledPaymentFlowOptions?.length === 1 &&
+						nonprofit?.metadata?.disabledPaymentFlowOptions[0]?.includes(method)
 					) {
 						return false;
 					}

packages/donate-button-v4/src/components/widget/types/Nonprofit.ts

@@ -18,9 +18,9 @@ export interface Nonprofit {
 	nteeCode: string | null;
 	metadata?: {
 		customTaxDeductible?: string;
-		// Note that multiple payment methods can be disabled, but it still arrives as a single
-		// comma-separated string from the API eg "card,paypal" so we type it as such here
-		disabledPaymentFlowOptions?: string;
+		// Note that multiple payment methods can be disabled, but for some reason it arrives
+		// as an array of length 1 eg ["card,paypal"] from the API, so we type it as such here
+		disabledPaymentFlowOptions?: string[];
 		disablePrivateNotes?: boolean;
 		granteeName?: string;
 		prefixWithThe?: boolean;