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

Priyanshu official/GOSW24 #75

Closed
wants to merge 9 commits into from
Closed

Priyanshu official/GOSW24 #75

wants to merge 9 commits into from

Conversation

Priyanshu-official
Copy link
Contributor

@Priyanshu-official Priyanshu-official commented Oct 29, 2020

Issue Number

ISSUE #55

Describe the changes you've made

Added Email notification feature after submitting the contact us form

Describe if there is any unusual behaviour of your code(Write NA if there isn't)

NA

Additional context (OPTIONAL)

I have Use Emailjs Which gives us More Feature and Easy to Implement
for Testing Create Id in Dashboard > https://dashboard.emailjs.com/

Adding Img of testing

Screenshot from 2020-10-29 13-19-16

Screenshot from 2020-10-29 13-19-24

Test plan (OPTIONAL)

A good test plan should give instructions that someone else can easily follow.
How someone can test your code?

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • The title of my pull request is a short description of the requested changes.

@Priyanshu-official Priyanshu-official changed the title Priyanshu official/55 Priyanshu official/GOSW24 Oct 29, 2020
@Priyanshu-official
Copy link
Contributor Author

@AVS18 Pls Review This

@geekyaditya100
Copy link
Contributor

Ok will check buddy

Copy link
Contributor

@07souravkunda 07souravkunda left a comment

Choose a reason for hiding this comment

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

Hi @Priyanshu-official ! EmailJS requires pricing.Can you set up an SMTP server with nodejs for email service.

@Priyanshu-official
Copy link
Contributor Author

@07souravkunda Ok I will do that with nodejs

@Priyanshu-official
Copy link
Contributor Author

@07souravkunda I am closing this I will Make new PR OF EMAIL
By implementation of NODEJS

@Priyanshu-official Priyanshu-official deleted the priyanshu-official/55 branch October 29, 2020 10:54
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.

3 participants