-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat: Design PayTM gateway modals #3447
Conversation
For each modal, just keep adding screenshots of them to the description 🖼 |
@mrsaicharan1 Yes, I'll be doing that |
bdd31c4
to
3d3a9ad
Compare
063a832
to
708f85c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we are going to add action in followup PR, this LGTM as a skeleton
@mrsaicharan1 @shreyanshdwivedi @kushthedude I have added the required validations in the modals. Please review now |
86b534d
to
9035566
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fixes #3446
Short description of what this resolves:
Add PayTM modal designs.
(actions will be linked once server PR has been completed)
Changes proposed in this pull request:
PayTM payment options modal created
![Screenshot from 2019-08-20 21-01-30](https://user-images.githubusercontent.com/21087061/63361558-faf9f100-c38d-11e9-979e-2d0eff60a3cf.png)
OTP entering modal prepared simultaneously
![Screenshot from 2019-08-20 21-04-13](https://user-images.githubusercontent.com/21087061/63361739-4c09e500-c38e-11e9-8559-88bbd9c6329c.png)
Checklist
development
branch.