Use proper array type (#447)

Author: markulrich

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

@@ -45,11 +45,8 @@ 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?.length === 1 &&
-						nonprofit?.metadata?.disabledPaymentFlowOptions[0]?.includes(method)
+						nonprofit?.metadata?.disabledPaymentFlowOptions?.includes(method)
 					) {
 						return false;
 					}

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

@@ -18,8 +18,6 @@ export interface Nonprofit {
 	nteeCode: string | null;
 	metadata?: {
 		customTaxDeductible?: 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;