Skip to content

fix: Issues in custom event name#19405

Merged
anikdhabal merged 4 commits intomainfrom
CAL-5086
Feb 21, 2025
Merged

fix: Issues in custom event name#19405
anikdhabal merged 4 commits intomainfrom
CAL-5086

Conversation

@anikdhabal
Copy link
Copy Markdown
Contributor

@anikdhabal anikdhabal commented Feb 20, 2025

@linear
Copy link
Copy Markdown

linear Bot commented Feb 20, 2025

@graphite-app graphite-app Bot requested a review from a team February 20, 2025 12:24
@keithwillcode keithwillcode added the core area: core, team members only label Feb 20, 2025
@dosubot dosubot Bot added event-types area: event types, event-types 🐛 bug Something isn't working labels Feb 20, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Feb 20, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (02/20/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (02/21/25)

1 label was added to this PR based on Keith Williams's automation.

@dosubot dosubot Bot added this to the v5.1 milestone Feb 20, 2025
@anikdhabal anikdhabal mentioned this pull request Feb 20, 2025
3 tasks
@linear
Copy link
Copy Markdown

linear Bot commented Feb 20, 2025

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 20, 2025

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

Name Status Preview Comments Updated (UTC)
cal-com-ui-playground ❌ Failed (Inspect) Feb 21, 2025 3:10pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Feb 21, 2025 3:10pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Feb 21, 2025 3:10pm

Comment on lines +155 to +164
<div
className={classNames(
"mb-2.5 flex font-normal",
customClassNames?.availableVariables?.variableItem?.container
)}>
<p
className={classNames(
"text-subtle ml-1 mr-5 w-32",
customClassNames?.availableVariables?.variableItem?.variableName
)}>
Copy link
Copy Markdown
Contributor Author

@anikdhabal anikdhabal Feb 20, 2025

Choose a reason for hiding this comment

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

Bring back Missing variables

Comment thread packages/core/event.ts Outdated
@calcom calcom deleted a comment from MehulZR Feb 20, 2025
@anikdhabal anikdhabal removed the Low priority Created by Linear-GitHub Sync label Feb 20, 2025
Comment thread packages/core/event.ts
Comment thread packages/core/event.ts
@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
* fix: Issues in custom event name

* Update event.ts
itsalam pushed a commit to itsalam/cal.com that referenced this pull request Mar 6, 2025
* fix: Issues in custom event name

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

Labels

🐛 bug Something isn't working core area: core, team members only event-types area: event types, event-types Low priority Created by Linear-GitHub Sync Medium priority Created by Linear-GitHub Sync ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-5086] Issues with Scheduler name variables in custom event name [CAL-4388] Variable {name} in "with event name" for the calendar does not work

4 participants