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

Contactus #327

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Contactus #327

wants to merge 7 commits into from

Conversation

akshay1027
Copy link
Contributor

@akshay1027 akshay1027 commented May 19, 2021

Description

Issue FIXES #185

Additional context (OPTIONAL)

Test plan (OPTIONAL)

Checklist

  • I have added my name in the contributors list at the end of README.md file.
  • 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 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.
  • Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

@vercel
Copy link

vercel bot commented May 19, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/feminine-divine/periodo/3cgm3n12HAVVTR7MiEpGWtoVSEvB
✅ Preview: Failed

[Deployment for 5fa9a44 failed]

@akshay1027
Copy link
Contributor Author

@Khushi-24 i think you guys have changed the form design while the setup of .env file and email js was going on! Now the problem arises as the new contact us design is totally different from what It was there when the PR was originally made.

@akshay1027
Copy link
Contributor Author

Let me know how to proceed! My suggestion would be to change to the old PR as it was given way before and then maybe we will again change the frontend alone to the new design that you guys have atm!

@akshay1027
Copy link
Contributor Author

Also i would suggest you please install email.js by yourself, only then we can stop the yarn.lock, package-lock file conflicts!

@khushishikhu
Copy link
Member

@akshay1027 Make it into the new design

@akshay1027
Copy link
Contributor Author

akshay1027 commented May 27, 2021

hey @khushishikhu am working as an intern, they are going to meet investors tomorrow and want the website to be ready by then! So am really busy working on that, henvce could not do this, if you can wait till tomorrow, I will finish this! Thanks and sorry for the Inconviences!

@Khushi-24
Copy link
Contributor

hey @Khushi-24 am working as an intern, they are going to meet investors tomorrow and want the website to be ready by then! So am really busy working on that, henvce could not do this, if you can wait till tomorrow, I will finish this! Thanks and sorry for the Inconviences!

@akshay1027 you are tagging the wrong person.

@akshay1027
Copy link
Contributor Author

hey @Khushi-24 am working as an intern, they are going to meet investors tomorrow and want the website to be ready by then! So am really busy working on that, henvce could not do this, if you can wait till tomorrow, I will finish this! Thanks and sorry for the Inconviences!

@akshay1027 you are tagging the wrong person.

Sorry!

@khushishikhu
Copy link
Member

@akshay1027 ohh it's fine. You can take your time.

@akshay1027
Copy link
Contributor Author

@khushishikhu i have made all the changes! please check it out! i cant resolve conflicts, I don't have the rights, also do install the email.js pack yourself to avoid confusions! Thank you

@akshay1027
Copy link
Contributor Author

@Khushi-24 whats the state of this PR, please let me know when you are free!

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.

Backend for contact us form
3 participants