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.

Sign up for GitHub

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

Jump to bottom

belindas-closet-nextjs_15_565_contact-us-page-dependency-update #566

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

belindas-closet-nextjs_15_565_contact-us-page-dependency-update #566

wants to merge 2 commits into from

Conversation

keiffer213
Copy link
Collaborator

Resolves #565

This PR updates the contact-us page to use the correct dependency for email forwarding

image

To test:

  1. run belindas-closet nextjs app locally
  2. navigate to contact-us page and fill out the form then press send
  3. Log in to emailjs dashboard using [email protected] and password Testing123!
  4. Navigate to email history and confirm that email has been sent

@keiffer213 keiffer213 added the bug Something isn't working label Dec 2, 2024
@keiffer213 keiffer213 self-assigned this Dec 2, 2024
@keiffer213 keiffer213 linked an issue Dec 2, 2024 that may be closed by this pull request
Contact-us-page dependency update #565
Open
@socket-security Socket Security
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/[email protected]

View full report↗︎

IsaacJrTypes
IsaacJrTypes approved these changes Dec 3, 2024
View reviewed changes
Copy link
Contributor

@IsaacJrTypes IsaacJrTypes left a comment

Choose a reason for hiding this comment

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

I successfully replicated the steps and the feature is working as intended. Good work!

ParysUnangst
ParysUnangst reviewed Dec 12, 2024
View reviewed changes
Copy link

@ParysUnangst ParysUnangst left a comment

Choose a reason for hiding this comment

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

I am receiving an error when hitting send
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ParysUnangst ParysUnangst ParysUnangst left review comments

@IsaacJrTypes IsaacJrTypes IsaacJrTypes approved these changes

@cshimm cshimm Awaiting requested review from cshimm

@nwm516 nwm516 Awaiting requested review from nwm516

@alasali1 alasali1 Awaiting requested review from alasali1

@JesseCaddell JesseCaddell Awaiting requested review from JesseCaddell

@miqo1385 miqo1385 Awaiting requested review from miqo1385

@el-riber el-riber Awaiting requested review from el-riber

@itsvee0120 itsvee0120 Awaiting requested review from itsvee0120

At least 2 approving reviews are required to merge this pull request.

Assignees

@keiffer213 keiffer213

Labels
bug Something isn't working
Projects
Belinda's Closet
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Contact-us-page dependency update
3 participants
@keiffer213 @IsaacJrTypes @ParysUnangst