Skip to content

Conversation

@STRRL
Copy link
Member

@STRRL STRRL commented Jul 2, 2025

  • introduce a new "contact us" page
  • introduce a floating button as "schedule a meeting"
image

design/style part might need adjustments, PTAL @g1eny0ung ❤️

STRRL added 2 commits July 2, 2025 14:40
Signed-off-by: Zhiqiang ZHOU <im@strrl.dev>
Signed-off-by: Zhiqiang ZHOU <im@strrl.dev>
@STRRL STRRL requested review from Copilot and g1eny0ung July 2, 2025 22:17
@chaotic-prow
Copy link

chaotic-prow bot commented Jul 2, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: STRRL

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new "Contact Us" page with an embedded Cal.com scheduler and a floating "Schedule a Meeting" button, and adds a corresponding nav link and script include.

  • Added static/js/cal-floating-button.js to load and configure a floating Cal.com widget
  • Created src/pages/contact.js for the Contact Us page, embedding an iframe scheduler and listing other channels
  • Updated docusaurus.config.js to include a "Contact Us" navbar item and load the floating button script

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
static/js/cal-floating-button.js New script for integrating a Cal.com floating button
src/pages/contact.js New Contact Us page with embedded scheduling iframe
docusaurus.config.js Added navbar link & script entry for floating button
Comments suppressed due to low confidence (3)

static/js/cal-floating-button.js:2

  • [nitpick] Variable names 'p', 'a', and 'ar' are ambiguous; consider renaming them to more descriptive names like 'enqueue', 'target', and 'args' to improve readability.
  let p = function (a, ar) {

static/js/cal-floating-button.js:1

  • [nitpick] Add a brief JSDoc or comment above this IIFE explaining its purpose and how it initializes the Cal.com embed script to aid future maintainers.
;(function (C, A, L) {

src/pages/contact.js:5

  • Consider adding unit or integration tests for the Contact page to ensure key elements (headings, iframe, links) render and behave as expected.
export default function Contact() {

@netlify
Copy link

netlify bot commented Jul 2, 2025

Deploy Preview for chaos-mesh-website-preview ready!

Name Link
🔨 Latest commit 7223176
🔍 Latest deploy log https://app.netlify.com/projects/chaos-mesh-website-preview/deploys/687783aa8231880008e7a621
😎 Deploy Preview https://deploy-preview-436--chaos-mesh-website-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Zhiqiang ZHOU <im@strrl.dev>
@STRRL
Copy link
Member Author

STRRL commented Jul 16, 2025

Hi @g1eny0ung I have updated the implementation with @calcom/embed-react, but there should be some polish on the styling I think <3

PTAL!

Signed-off-by: Yue Yang <g1enyy0ung@gmail.com>
@g1eny0ung
Copy link
Member

@STRRL I have also updated the style. PTAL :)

@STRRL
Copy link
Member Author

STRRL commented Jul 16, 2025

that's cool! LGTM!

I think we could merge this PR now?

@g1eny0ung
Copy link
Member

Sure!

@g1eny0ung g1eny0ung merged commit 21909e0 into chaos-mesh:master Jul 17, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants