Skip to content

test: email verification flow#9437

Closed
shivamklr wants to merge 9 commits intocalcom:mainfrom
shivamklr:test/emailVerificationFlow
Closed

test: email verification flow#9437
shivamklr wants to merge 9 commits intocalcom:mainfrom
shivamklr:test/emailVerificationFlow

Conversation

@shivamklr
Copy link
Copy Markdown
Contributor

@shivamklr shivamklr commented Jun 9, 2023

What does this PR do?

Fixes #9436
Test #9081

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Test (E2E Test)
  • This change requires a documentation update

How should this be tested?

  • Test A
  • Test B

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my PR needs changes to the documentation
  • I haven't checked if my changes generate no new warnings
  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 12, 2023 1:04pm

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 9, 2023

@shivamklr is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added 🐛 bug Something isn't working ❗️ .env changes contains changes to env variables labels Jun 9, 2023
@socket-security
Copy link
Copy Markdown

New and updated dependency changes detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives1 Size Publisher
mailhog 🆕 4.16.0 network +0 51.2 kB blueimp

Footnotes

  1. https://docs.socket.dev

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 9, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@shivamklr shivamklr changed the title Test/email verification flow test: email verification flow Jun 9, 2023
@socket-security
Copy link
Copy Markdown

🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎

To accept the risk, merge this PR and you will not be notified again.

Issue Package Version Note Source
Network access mailhog 4.16.0 package.json

Next steps

What is network access?

This module accesses the network.

Packages should remove all network access that isn't functionally unnecessary. Consumers should audit network access to ensure legitimate use.

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@* or ignore all packages with @SocketSecurity ignore-all

  • @SocketSecurity ignore mailhog@4.16.0

@shivamklr
Copy link
Copy Markdown
Contributor Author

I pick this up in a few days. Occupied with something urgent at the moment.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 4, 2023

This PR is being marked as stale due to inactivity.

@github-actions github-actions bot added the Stale label Jul 4, 2023
@github-actions
Copy link
Copy Markdown
Contributor

This PR is being closed due to inactivity. Please reopen if work is intended to be continued.

@github-actions github-actions bot closed this Jul 12, 2023
@tadinski
Copy link
Copy Markdown

is this issue abandoned?

@shivamklr shivamklr deleted the test/emailVerificationFlow branch August 2, 2023 07:31
@nicktrn nicktrn mentioned this pull request Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working ❗️ .env changes contains changes to env variables Stale

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

sendEmail Error: Missing Credentials for "PLAIN"

2 participants