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

revert: Reapply "fix: Multiple rr-hosts combine to create erroneous availabil… #19263

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

revert: Reapply "fix: Multiple rr-hosts combine to create erroneous availabil… #19263

wants to merge 8 commits into from
+788 −196

Conversation

emrysal
Copy link
Contributor

@emrysal emrysal commented Feb 12, 2025

…ity (#18772)"

This reverts commit 5dbc6d9.

What does this PR do?

Temporarily duplicate slot generation logic to do production A/B testing

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 12, 2025 •
edited
Loading

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "Reapply "fix: Multiple rr-hosts combine to create erroneous availabil…". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@keithwillcode keithwillcode added core area: core, team members only foundation labels Feb 12, 2025
@vercel Vercel
Copy link

vercel bot commented Feb 12, 2025 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
cal-com-ui-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 13, 2025 0:42am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 0:42am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Feb 13, 2025 0:42am

@emrysal emrysal marked this pull request as ready for review February 13, 2025 12:38
@emrysal emrysal requested a review from a team as a code owner February 13, 2025 12:38
@dosubot dosubot bot added the bookings area: bookings, availability, timezones, double booking label Feb 13, 2025
@emrysal emrysal changed the title Reapply "fix: Multiple rr-hosts combine to create erroneous availabil… revert: Reapply "fix: Multiple rr-hosts combine to create erroneous availabil… Feb 13, 2025
@github-actions GitHub Actions
Copy link
Contributor

E2E results are ready!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bookings area: bookings, availability, timezones, double booking core area: core, team members only foundation ready-for-e2e
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@emrysal @keithwillcode