Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve UX for setting up Payment Methods. #10675

Merged
merged 1 commit into from Jan 13, 2021
Merged

Improve UX for setting up Payment Methods. #10675

merged 1 commit into from Jan 13, 2021

Conversation

MatthewKennedy
Copy link
Contributor

@MatthewKennedy MatthewKennedy commented Jan 13, 2021

Moved Payment Provider setting into separate cards (similar to shipping method settings)

One card for the default settings that come with every payment method, and a card for payment provider specific settings. This allows users to become familiar with the standard settings and instantly recognise custom fields as they are provided.

Reason for PR:
Personally I found when switching payment providers it was sometimes a spot-the-difference moment when you updated the provider to see what fields had just been inserted amongst the regular fields.

No Preferences needed:
If no settings are available a message is displayed.
Screenshot 2021-01-13 at 11 05 01

Custom Preferences offered:
Screenshot 2021-01-13 at 11 05 29

@squash-labs
Copy link

squash-labs bot commented Jan 13, 2021

Manage this branch in Squash

Test this branch here: https://matthewkennedyuser-friendly-pa-z37kx.squash.io

Copy link
Member

@damianlegawiec damianlegawiec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really nice, thank you @MatthewKennedy 👍

@damianlegawiec damianlegawiec merged commit 48348ea into spree:master Jan 13, 2021
@MatthewKennedy MatthewKennedy deleted the user-friendly-payment-methods branch January 13, 2021 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants