Updated Contact Us Page to Use Form#264
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| @@ -1,4 +1,3 @@ | |||
| import React from 'react' | |||
There was a problem hiding this comment.
oh.... didn't realize it wasn't used anymore haha thanks
| } | ||
|
|
||
| const { data, error } = await resend.emails.send({ | ||
| from: 'Teacher Fund <joel@theteacherfund.com>', |
There was a problem hiding this comment.
shouldn't this be sending an email from them to me, instead of the other way around?
There was a problem hiding this comment.
Hehehe. My bad.
Let me fix.
I can't remember when I made the typo.. haha
There was a problem hiding this comment.
I think I remember why I wrote the from like that, but I forgot to update the to. Because we are using Resend, emails cannot be sent from unverified email domain, and the funder's email domain is likely unverified. I have updated it, what do you think?
joelwass
left a comment
joelwass
left a comment
There was a problem hiding this comment.
i haven't been able to test because for some reason api/contact in the vercel preview is 404'ing, but once you address the "to" and "from" swap it looks good to me and i'll merge it and test it locally :)
…nd_next into dev/update-contact-page-to-use-form
| Separator, | ||
| CardBody, | ||
| Badge | ||
| } from '@chakra-ui/react' |
There was a problem hiding this comment.
i don't think we're going to be able to use chakra here without some hacking. i couldn't figure it out in the 10 mins i gave to this.
you need a chakra provider, and i believe that Resend is rendering this component outside of the chakra provider.
i'll dm you the resend API key so you can test locally
…nd_next into dev/update-contact-page-to-use-form
… the email templates
2 participants
In this PR, I revamped the contact page to play out as suggested in issue #188. There is an existing PR under this issue, #203, which failed to sync with the version of Chakra used in the codebase.
Here is what this PR contains:
Thanks.