Skip to content

chore: remove unused TokenHandler component - #30082

Closed
enesbugrahankilic wants to merge 1 commit into
calcom:mainfrom
enesbugrahankilic:chore/remove-unused-token-handler
Closed

chore: remove unused TokenHandler component#30082
enesbugrahankilic wants to merge 1 commit into
calcom:mainfrom
enesbugrahankilic:chore/remove-unused-token-handler

Conversation

@enesbugrahankilic

@enesbugrahankilic enesbugrahankilic commented Aug 30, 2026

Copy link
Copy Markdown

Summary

  • Removes the unused TokenHandler component and its test file from packages/ui/components/TokenHandler/
  • The component was not exported from the UI package index and had no imports elsewhere in the codebase

Fixes #30077

Test plan

  • Confirmed no remaining references to TokenHandler in the repo
  • Only the component's own test file referenced it

@github-actions

Copy link
Copy Markdown
Contributor

Welcome to Cal.diy, @enesbugrahankilic! Thanks for opening this pull request.

A few things to keep in mind:

  • This is Cal.diy, not Cal.com. Cal.diy is a community-driven, fully open-source fork of Cal.com licensed under MIT. Your changes here will be part of Cal.diy — they will not be deployed to the Cal.com production app.
  • Please review our Contributing Guidelines if you haven't already.
  • Make sure your PR title follows the Conventional Commits format.

A maintainer will review your PR soon. Thanks for contributing!

@github-actions github-actions Bot added the 🐛 bug Something isn't working label Aug 30, 2026
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7dbe821b-98ab-49e9-a493-96085dff2d17

📥 Commits

Reviewing files that changed from the base of the PR and between 176037d and c37e128.

📒 Files selected for processing (2)
  • packages/ui/components/TokenHandler/TokenHandler.tsx
  • packages/ui/components/TokenHandler/token-handler.test.tsx
💤 Files with no reviewable changes (2)
  • packages/ui/components/TokenHandler/TokenHandler.tsx
  • packages/ui/components/TokenHandler/token-handler.test.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

Removed the unused TokenHandler React component and its related PropType definition. Removed tests for input count rendering and digit-change handling.

Merge Risk: ⚪ Minimal · up to c37e1

This change removes an unused UI component and its test without altering active product behavior. No actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The pull request removes both files required by issue #30077: TokenHandler.tsx and token-handler.test.tsx. This matches the issue objective to remove unused code that has no external references.
Out of Scope Changes check ✅ Passed The changes are limited to deleting the unused TokenHandler component and its test file. No unrelated changes are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Title check ✅ Passed The title clearly and concisely describes the main change: removal of the unused TokenHandler component.
Description check ✅ Passed The description directly explains the removal of the unused TokenHandler component and its test file, including the reason and verification steps.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@enesbugrahankilic

Copy link
Copy Markdown
Author

Hi! This is ready for review whenever convenient. Could a maintainer add the run-ci label so CI can run? Thanks!

The TokenHandler component is not exported or imported anywhere in the
codebase. Removes the dead code and its test file.

Fixes calcom#30077

Co-authored-by: Cursor <cursoragent@cursor.com>
@enesbugrahankilic
enesbugrahankilic force-pushed the chore/remove-unused-token-handler branch from c37e128 to 3f0478b Compare August 31, 2026 06:11
@bandhan-majumder

Copy link
Copy Markdown
Member

closing as fixed by: #30084

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

Labels

🐛 bug Something isn't working size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: remove unused TokenHandler component

2 participants