-
Notifications
You must be signed in to change notification settings - Fork 35
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
Send email after Certificate Generation #35
Labels
Comments
@vigasdeep @mandeeps708 When the user will enter mail. Do we verify that mail or directly send link to the mail. |
On more thing. If we will send the link of that Certificate to him immediately then why we need to store his email in database. |
for future use.
…On Tue, Oct 10, 2017 at 4:15 PM, Amisha Budhiraja ***@***.***> wrote:
On more thing. If we will send the link of that Certificate to him
immediately then why we need to store his email in database.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#35 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABCzTXLQ2DeyUjfhNVODOSifk6URMkx4ks5sq0q5gaJpZM4Pxcn8>
.
--
Vigas Deep
[W] | https://vigasdeep.com
[M] | +91-9815-400-807
|
Not verify by sending mail. Just validate that it is actual email, using
the field type.
…On Tue, Oct 10, 2017 at 4:12 PM, Amisha Budhiraja ***@***.***> wrote:
@vigasdeep <https://github.com/vigasdeep> @mandeeps708
<https://github.com/mandeeps708> When the user will enter mail. Do we
verify that mail or directly send link to the mail.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#35 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABCzTUfRnDBjHtiWOXSlJMTIrIDwMY1Zks5sq0ovgaJpZM4Pxcn8>
.
--
Vigas Deep
[W] | https://vigasdeep.com
[M] | +91-9815-400-807
|
Okay. Thankyou for quick reply. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Here is TTD for this issue
To Discuss: We may need to use database now, what others think?
The text was updated successfully, but these errors were encountered: