- Issue created by @rszrama
- πΊπ¦Ukraine marchuk.vitaliy Rivne, UA
vmarchuk β made their first commit to this issueβs fork.
- last update
over 1 year ago 37 pass, 2 fail - @vmarchuk opened merge request.
- last update
over 1 year ago 37 pass, 2 fail - last update
over 1 year ago 38 pass - Status changed to Needs review
over 1 year ago 2:51pm 9 October 2023 - Status changed to Needs work
over 1 year ago 3:44am 11 October 2023 - πΊπΈUnited States rszrama
I've reviewed this and think it needs a little work.
Ultimately, I think we should not keep the "Enable credit card icons" field on the form. It's confusing to have to check that checkbox in order to activate showing payment method logos, even though "No" is an option in the select list. In other words, when the form is rendered and there is a value for the old setting, it should be used to set the values of the new settings in an equivalent fashion ... but we shouldn't be trying to preserve and interpret the checkbox after this update.
- last update
over 1 year ago 38 pass - last update
over 1 year ago 38 pass - Status changed to Needs review
over 1 year ago 8:21am 11 October 2023 - πΊπ¦Ukraine marchuk.vitaliy Rivne, UA
@rszrama
Ultimately, I think we should not keep the "Enable credit card icons" field on the form. It's confusing to have to check that checkbox in order to activate showing payment method logos, even though "No" is an option in the select list. In other words, when the form is rendered and there is a value for the old setting, it should be used to set the values of the new settings in an equivalent fashion ... but we shouldn't be trying to preserve and interpret the checkbox after this update.
It's done. I also updated the settings to set the default credit cards for backwards compatibility. This means that if someone after updating the module opens the payment gateway settings form and then just clicks "Save", we will keep the existing behavior but with the new settings.