Skip to content

fix: Adding more time to org admin email verif step#9574

Merged
emrysal merged 1 commit intomainfrom
chore/orgs-admin-email-verif
Jun 16, 2023
Merged

fix: Adding more time to org admin email verif step#9574
emrysal merged 1 commit intomainfrom
chore/orgs-admin-email-verif

Conversation

@leog
Copy link
Copy Markdown
Contributor

@leog leog commented Jun 16, 2023

What does this PR do?

Making the organization admin email verification code sent valid for 15 minutes. It was 90 seconds, not enough for people to get the email and use the code successfully.

Type of change

  • Chore (refactoring code, technical debt, workflow improvements)

How should this be tested?

When creating an org, the step to verify your email will send you an email with a code. That code should be valid for 15 minutes.

Mandatory Tasks

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

@leog leog added the organizations area: organizations, orgs label Jun 16, 2023
@leog leog requested a review from a team June 16, 2023 14:26
@leog leog self-assigned this Jun 16, 2023
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 16, 2023

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

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2023 2:31pm
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2023 2:31pm
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2023 2:31pm
web-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2023 2:31pm

@github-actions
Copy link
Copy Markdown
Contributor

📦 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! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jun 16, 2023

Current Playwright Test Results Summary

✅ 98 Passing - ⚠️ 3 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 06/16/2023 02:38:00pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: dbab6f3

Started: 06/16/2023 02:36:15pm UTC

⚠️ Flakes

📄   apps/web/playwright/login.2fa.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
2FA Tests should allow a user to enable 2FA and login using 2FA
Retry 1Initial Attempt
4.83% (13) 13 / 269 runs
failed over last 7 days
15.99% (43) 43 / 269 runs
flaked over last 7 days

📄   apps/web/playwright/login.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
user can login & logout succesfully login flow user & logout using dashboard
Retry 1Initial Attempt
5.82% (16) 16 / 275 runs
failed over last 7 days
28.73% (79) 79 / 275 runs
flaked over last 7 days

📄   apps/web/playwright/change-password.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Change Password Test change password
Retry 1Initial Attempt
0% (0) 0 / 263 runs
failed over last 7 days
1.14% (3) 3 / 263 runs
flaked over last 7 days

View Detailed Build Results


@emrysal emrysal merged commit 8fa1c70 into main Jun 16, 2023
@emrysal emrysal deleted the chore/orgs-admin-email-verif branch June 16, 2023 14:38
iamr-kumar pushed a commit to iamr-kumar/cal.com that referenced this pull request Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

organizations area: organizations, orgs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants