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

feat(webapp): allow non kyc #366

Merged
merged 11 commits into from
Jan 19, 2022
Merged

feat(webapp): allow non kyc #366

merged 11 commits into from
Jan 19, 2022

Conversation

leisterfrancisco
Copy link
Collaborator

Update user application flow

What does this PR do?

  • Allow non-kyc user to apply

Steps to test

  1. make run

@leisterfrancisco
Copy link
Collaborator Author

Final mail

image

@lgtm-com
Copy link

lgtm-com bot commented Jan 19, 2022

This pull request fixes 1 alert when merging 6e88e83 into 523b734 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

</p>
<p align="left" style="flex-grow: 0; margin: 0px 16px 14px; font-family: Arial; font-size: 14px; font-weight: normal; font-stretch: normal; font-style: normal; line-height: 1.14; letter-spacing: 0.44px; text-align: center; color: #fff;">
<a href="https://forms.gle/GWHig5ciAvg5fdEH7" style="color: #fff;">
Apply Here for Funding
Copy link
Contributor

Choose a reason for hiding this comment

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

does this need to be in the email?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is coming from a template that a member of the Proton Affiliate team gave us.

Copy link
Contributor

@tetogomez tetogomez left a comment

Choose a reason for hiding this comment

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

LGTM

@lgtm-com
Copy link

lgtm-com bot commented Jan 19, 2022

This pull request fixes 1 alert when merging 1e92cfb into 9da0196 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@leisterfrancisco
Copy link
Collaborator Author

After comments

image

@lgtm-com
Copy link

lgtm-com bot commented Jan 19, 2022

This pull request fixes 1 alert when merging f7c5371 into 9da0196 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@xavier506 xavier506 merged commit 97fe060 into dev Jan 19, 2022
@xavier506 xavier506 deleted the feat/allow-non-kyc branch January 19, 2022 17:46
@leisterfrancisco leisterfrancisco restored the feat/allow-non-kyc branch January 19, 2022 17:48
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

3 participants