Skip to content

Conversation

@Akkii88
Copy link

@Akkii88 Akkii88 commented Jan 27, 2026

Fixes GH-27261

Corrected timezone resolution logic for booking limits. The system now prioritizes the Event Type timezone, then the Schedule timezone, and finally the User timezone (Organizer) to determine the "day" boundary. This prevents bookings on different local days (but same UTC day) from being incorrectly blocked by daily limits.

@Akkii88 Akkii88 requested review from a team as code owners January 27, 2026 10:57
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link
Contributor

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 "Fix/booking limit gh 27261". 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

@github-actions github-actions bot added consumer High priority Created by Linear-GitHub Sync 🐛 bug Something isn't working labels Jan 27, 2026
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Jan 27, 2026
@graphite-app
Copy link

graphite-app bot commented Jan 27, 2026

Graphite Automations

"Send notification to Community team when High Priority PR opened" took an action on this PR • (01/27/26)

2 teammates were notified to this PR based on Keith Williams's automation.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 7 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="packages/features/tsconfig.json">

<violation number="1" location="packages/features/tsconfig.json:27">
P2: Test files no longer excluded from the features tsconfig; `**/*.test.*` removal causes non-__tests__ test files to be compiled/typechecked in builds</violation>
</file>

Since this is your first cubic review, here's how it works:

  • cubic automatically reviews your code and comments on bugs and improvements
  • Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
  • Ask questions if you need clarification on any suggestion

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

"exclude": [
"dist",
"build",
"**/node_modules/**",
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Jan 27, 2026

Choose a reason for hiding this comment

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

P2: Test files no longer excluded from the features tsconfig; **/*.test.* removal causes non-tests test files to be compiled/typechecked in builds

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/features/tsconfig.json, line 27:

<comment>Test files no longer excluded from the features tsconfig; `**/*.test.*` removal causes non-__tests__ test files to be compiled/typechecked in builds</comment>

<file context>
@@ -5,12 +5,29 @@
+  "exclude": [
+    "dist",
+    "build",
+    "**/node_modules/**",
+    "**/__mocks__/*",
+    "**/__tests__/*",
</file context>
Fix with Cubic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working community Created by Linear-GitHub Sync consumer High priority Created by Linear-GitHub Sync size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Could not book the meeting - booking limit has been reached

2 participants