Skip to content

chore: remove unused queries on user event types#11159

Merged
keithwillcode merged 2 commits intomainfrom
fix-unused-queries-event-types-get-by-viewer
Sep 6, 2023
Merged

chore: remove unused queries on user event types#11159
keithwillcode merged 2 commits intomainfrom
fix-unused-queries-event-types-get-by-viewer

Conversation

@alannnc
Copy link
Copy Markdown
Contributor

@alannnc alannnc commented Sep 5, 2023

What does this PR do?

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is ab UI/UX design document, please, share it here.

Type of change

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

How should this be tested?

  • Are there environment variables that should be set? No
  • What are the minimal test data to have? Any test user with own event types and team event types
  • What is expected (happy path) to have (input and output)? Managed event
  • Any other important info that could help to test that PR

Mandatory Tasks

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

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 5, 2023

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

Name Status Preview Comments Updated (UTC)
ai ❌ Failed (Inspect) Sep 6, 2023 0:46am
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 6, 2023 0:46am
cal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 6, 2023 0:46am
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 6, 2023 0:46am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 6, 2023 0:46am
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Sep 6, 2023 0:46am
qa ⬜️ Ignored (Inspect) Visit Preview Sep 6, 2023 0:46am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2023

Thank you for following the naming conventions! 🙏

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 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! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Sep 5, 2023

Current Playwright Test Results Summary

✅ 117 Passing - ⚠️ 3 Flaky

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

(Last updated on 09/06/2023 12:45:14am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 96a2a54

Started: 09/06/2023 12:42:46am 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
0.75% (2) 2 / 266 runs
failed over last 7 days
27.82% (74) 74 / 266 runs
flaked over last 7 days

📄   apps/web/playwright/integrations-stripe.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Stripe integration When event is paid and confirmed Cancelled paid booking should be refunded
Retry 1Initial Attempt
0.76% (2) 2 / 263 runs
failed over last 7 days
6.46% (17) 17 / 263 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 1Initial Attempt
0.72% (2) 2 / 276 runs
failed over last 7 days
97.83% (270) 270 / 276 runs
flaked over last 7 days

View Detailed Build Results


@keithwillcode keithwillcode merged commit 829f90b into main Sep 6, 2023
@keithwillcode keithwillcode deleted the fix-unused-queries-event-types-get-by-viewer branch September 6, 2023 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants