Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

fix: One flaky E2E tests#24815

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
anikdhabal merged 16 commits intomainfromdevin/exclude-seated-responses-1761905164
Nov 20, 2025

Conversation

@anikdhabal
Copy link
Contributor

@anikdhabalanikdhabal commentedOct 31, 2025•
edited
Loading

What does this PR do?

…ted eventsFor seated events (events with seatsPerTimeSlot), booking responses are now excluded from being sent to calendar integrations (Google Calendar, etc.), CRM integrations (HubSpot, Salesforce, etc.), and other integrations.This applies to both creating new seated bookings and rescheduling seated bookings.The calendar event and CRM records are still created, but without the custom booking field responses included in the description/notes.Co-Authored-By: anik@cal.com <adhabal2002@gmail.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel
Copy link

vercelbot commentedNov 18, 2025•
edited
Loading

The latest updates on your projects. Learn more aboutVercel for GitHub.

2 Skipped Deployments
ProjectDeploymentPreviewCommentsUpdated (UTC)
calIgnoredIgnoredNov 20, 2025 10:11am
cal-euIgnoredIgnoredNov 20, 2025 10:11am

Removed debug logging for redirect URI and client ID.
@github-actions
Copy link
Contributor

github-actionsbot commentedNov 18, 2025•
edited
Loading

E2E results are ready!

Refactor database connection handling for integration tests.
@anikdhabalanikdhabal changed the titlefix: exclude booking responses from calendar/CRM integrations for seated eventsfix: flaky E2E testsNov 19, 2025
@anikdhabalanikdhabal changed the titlefix: flaky E2E testsfix: One flaky E2E testsNov 20, 2025
@anikdhabalanikdhabal marked this pull request as ready for reviewNovember 20, 2025 16:29
@anikdhabalanikdhabalenabled auto-merge (squash)November 20, 2025 16:29
Copy link
Contributor

@cubic-dev-aicubic-dev-aibot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@anikdhabalanikdhabal merged commit176fb77 intomainNov 20, 2025
40 of 41 checks passed
@anikdhabalanikdhabal deleted the devin/exclude-seated-responses-1761905164 branchNovember 20, 2025 16:40
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@cubic-dev-aicubic-dev-ai[bot]cubic-dev-ai[bot] left review comments

@dhairyashiildhairyashiildhairyashiil approved these changes

Assignees

No one assigned

Labels

corearea: core, team members onlyready-for-e2esize/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@anikdhabal@dhairyashiil@keithwillcode
[8]ページ先頭
©2009-2025 Movatter.jp