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

Save payment info #2164

Closed
dotmacro opened this issue Jul 8, 2024 · 0 comments · Fixed by #2268
Closed

Save payment info #2164

dotmacro opened this issue Jul 8, 2024 · 0 comments · Fixed by #2268

Comments

@dotmacro
Copy link
Member

dotmacro commented Jul 8, 2024

Problem

User's payment info is lost when changing from below mincome to pledging.

  1. User does not make mincome, enters payment info.
paymentsBelow
  1. User now makes mincome, adjusts income details accordingly. Payment info is removed from UI.
paymentsAbove
  1. User does not make mincome, adjusts income details accordingly. User needs to re-fill in all payment info.
paymentsBelow2

Solution

Store payment info so user does not have to manually re-enter each time they switch income details to above/below mincome. This will provide better UX (and also help with payments matching, if that's added to the app).

@SebinSong SebinSong self-assigned this Jul 10, 2024
taoeffect pushed a commit that referenced this issue Aug 6, 2024
#2268)

* do not clear the saved payment info

* must work when a field is removed too

* check after general validations

* flaky test

* comment update agin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants