Skip to content

feat: advanced tab platform wrapper#16941

Merged
ThyMinimalDev merged 13 commits intomainfrom
event-advanced-platform-wrapper
Oct 10, 2024
Merged

feat: advanced tab platform wrapper#16941
ThyMinimalDev merged 13 commits intomainfrom
event-advanced-platform-wrapper

Conversation

@ibex088
Copy link
Copy Markdown
Contributor

@ibex088 ibex088 commented Oct 4, 2024

-- used DOMPurify instead of sanitize-html

What does this PR do?

Screenshot 2024-10-04 at 4 13 26 PM

  • Fixes #XXXX (GitHub issue number)
  • Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have added a Docs issue here if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings

-- used `DOMPurify` instead of `sanitize-html`
@graphite-app graphite-app Bot requested review from a team October 4, 2024 10:46
@dosubot dosubot Bot added platform Anything related to our platform plan ✨ feature New feature or request labels Oct 4, 2024
@keithwillcode keithwillcode added the core area: core, team members only label Oct 4, 2024
@graphite-app graphite-app Bot requested a review from a team October 4, 2024 10:47
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Oct 4, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (10/04/24)

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

"Add foundation team as reviewer" took an action on this PR • (10/04/24)

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

"Add platform team as reviewer" took an action on this PR • (10/04/24)

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

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 7, 2024

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

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Oct 10, 2024 1:42pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Oct 10, 2024 1:42pm

Comment thread packages/features/eventtypes/components/tabs/advanced/EventAdvancedTab.tsx Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 10, 2024

E2E results are ready!

@socket-security
Copy link
Copy Markdown

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/@formkit/auto-animate@1.0.0-beta.5

View full report↗︎

@ThyMinimalDev ThyMinimalDev merged commit 99caae8 into main Oct 10, 2024
@ThyMinimalDev ThyMinimalDev deleted the event-advanced-platform-wrapper branch October 10, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only ✨ feature New feature or request platform Anything related to our platform plan ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants