feat: Plain chat #18284

nizzyabi · 2024-12-20T03:58:49Z

What does this PR do?

This PR replaces the main intercom chat with plain.com's live chat feature.

Mandatory Tasks (DO NOT REMOVE)

[Y ] I have self-reviewed the code (A decent size PR without self-review might be rejected).
[ Y] I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
[ Y] 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?
PLAIN_CHAT_ID=
PLAIN_CHAT_HMAC_SECRET_KEY=
What are the minimal test data to have?
Post requests to plain.com
What is expected (happy path) to have (input and output)?
Able to see user messages in plain
Any other important info that could help to test that PR

Checklist

github-actions · 2024-12-20T03:59:03Z

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 "Plain chat WIP". 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

graphite-app · 2024-12-20T04:00:11Z

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (12/20/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 • (12/20/24)

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

"Add ready-for-e2e label" took an action on this PR • (12/26/24)

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

vercel · 2024-12-20T16:27:00Z

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		Dec 26, 2024 6:22pm
calcom-web-canary	⬜️ Ignored (Inspect)	Visit Preview		Dec 26, 2024 6:22pm

socket-security · 2024-12-20T16:33:21Z

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

…to /support

hbjORbj · 2024-12-20T23:05:21Z

@nizzyabi Type-check test is failing! You can check https://github.com/calcom/cal.com/actions/runs/12439539908/job/34733552858?pr=18284

emrysal · 2024-12-26T16:18:53Z

apps/web/app/api/customer-card/route.ts

-                          : customer.emailVerified
-                          ? "GREEN"
-                          : "RED",
+                        customer.emailVerified === undefined ? "No" : customer.emailVerified ? "Yes" : "No",


Suggested change

customer.emailVerified === undefined ? "No" : customer.emailVerified ? "Yes" : "No",

customer.emailVerified ? "Yes" : "No",

emrysal · 2024-12-26T16:22:54Z

apps/web/lib/plain/plainChat.tsx

+    }
+  `;
+
+  useEffect(() => {


This duplicates the hook logic; what do you say about moving this into the hook entirely?

apps/web/lib/plain/plainChat.tsx

Co-authored-by: Syed Ali Shahbaz <[email protected]>

…pp router metadata logic (#18348) * remove HeadSeo for already migrated pages and refactor prepareMetadata * create _generateMetadataWithoutImage and refactor _generateMetadata

* chore: app router - /bookings page * remove env vars * fix * Update middleware.ts * revert unneeded change * refactor for the better * fix

Co-authored-by: CarinaWolli <[email protected]> Co-authored-by: sean-brydon <[email protected]> Co-authored-by: sean-brydon <[email protected]> Co-authored-by: Peer Richelsen <[email protected]> Co-authored-by: Omar López <[email protected]>

Co-authored-by: Replexica <[email protected]>

…g parent (#18059) * fix: metadata is overwirten * Update * type error * Update * fix test * fix type error

* Added routing form support link * small change * Type fix --------- Co-authored-by: Joe Au-Yeung <[email protected]>

* refactor handling logic for embeds in app router * fix type checks * add test for withEmbedSsrAppDir * fix

CLAassistant · 2024-12-26T18:11:34Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
6 out of 7 committers have signed the CLA.

✅ nizzyabi
✅ Udit-takkar
✅ vijayraghav-io
✅ hbjORbj
✅ emrysal
✅ anikdhabal
❌ calcom-bot
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

emrysal

lgtm

github-actions · 2024-12-26T18:58:33Z

E2E results are ready!

Workflow #42394.2 latest results

graphite-app bot requested review from a team December 20, 2024 03:58

keithwillcode added the core area: core, team members only label Dec 20, 2024

vercel bot deployed to Preview – api December 20, 2024 04:05 View deployment

github-actions bot added the ❗️ .env changes contains changes to env variables label Dec 20, 2024

vercel bot deployed to Preview – api December 20, 2024 16:33 View deployment

vercel bot deployed to Preview – api December 20, 2024 19:11 View deployment

nizzyabi added 5 commits December 20, 2024 15:58

plain custom desin

8c812f5

No display on card

60bba1f

dynamic plain chat component and hmac hash

e9ca446

re-route users to plain.com chat instead of intercom chat when going …

064e8d9

…to /support

provider errors

9fbf3f1

nizzyabi force-pushed the plain-chat branch from 628ee16 to 9fbf3f1 Compare December 20, 2024 20:59

yarn lock fix

9faad04

vercel bot deployed to Preview – api December 20, 2024 21:15 View deployment

plain chat removed unneeded hmac

6138296

vercel bot deployed to Preview – api December 20, 2024 22:52 View deployment

hbjORbj changed the title ~~Plain chat WIP~~ feat: Plain chat WIP Dec 20, 2024

hbjORbj marked this pull request as draft December 20, 2024 23:04

hbjORbj changed the title ~~feat: Plain chat WIP~~ feat: Plain chat Dec 20, 2024

fix ts error

ee834d6

vercel bot deployed to Preview – api December 20, 2024 23:55 View deployment

nizzyabi added 2 commits December 20, 2024 20:11

error handling improved

127af7d

remove intercome provider from app-dir

e1b40ea

vercel bot deployed to Preview – api December 21, 2024 01:22 View deployment

dosubot bot added the ✨ feature New feature or request label Dec 26, 2024

emrysal reviewed Dec 26, 2024

View reviewed changes

apps/web/lib/plain/plainChat.tsx Show resolved Hide resolved

emrysal reviewed Dec 26, 2024

View reviewed changes

apps/web/lib/plain/plainChat.tsx Show resolved Hide resolved

PeerRich and others added 15 commits December 26, 2024 13:11

Create getting-started.mdx (#18342)

9e2196f

Delete help directory (#18343)

2abe4fe

chore: moved docs/help to /help (#18345)

2e43fd8

chore: Delete unused guides directory (#18346)

84a2300

feat: booking filters (#18303)

2bba876

Co-authored-by: Syed Ali Shahbaz <[email protected]>

chore: Remove HeadSeo components where no longer needed + improve a…

468a704

…pp router metadata logic (#18348) * remove HeadSeo for already migrated pages and refactor prepareMetadata * create _generateMetadataWithoutImage and refactor _generateMetadata

chore: app router - /bookings status page (#18183)

6581aca

* chore: app router - /bookings page * remove env vars * fix * Update middleware.ts * revert unneeded change * refactor for the better * fix

cache i18n instances (#18309)

4810649

feat: update translations via @replexica (#18361)

8c81e13

Co-authored-by: Replexica <[email protected]>

update OOO e2e tests to remove flakiness (#18367)

291e58b

fix: metadata is overwritten for child managed eventType when updatin…

e279d02

…g parent (#18059) * fix: metadata is overwirten * Update * type error * Update * fix test * fix type error

chore: added routing support link (#18369)

08a9a75

* Added routing form support link * small change * Type fix --------- Co-authored-by: Joe Au-Yeung <[email protected]>

chore: refactor handling logic for embeds in app router (#18362)

ab7d481

* refactor handling logic for embeds in app router * fix type checks * add test for withEmbedSsrAppDir * fix

review changes

b02ec98

nizzyabi and others added 2 commits December 26, 2024 13:11

yarn lock

3866ca0

Merge branch 'main' into plain-chat

6068fb4

vercel bot deployed to Preview – api December 26, 2024 18:33 View deployment

emrysal approved these changes Dec 26, 2024

View reviewed changes

graphite-app bot added the ready-for-e2e label Dec 26, 2024

emrysal enabled auto-merge (squash) December 26, 2024 18:36

emrysal merged commit 225313e into main Dec 26, 2024
58 of 60 checks passed

emrysal deleted the plain-chat branch December 26, 2024 19:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Plain chat #18284

feat: Plain chat #18284

nizzyabi commented Dec 20, 2024 •

edited

Loading

github-actions bot commented Dec 20, 2024 •

edited

Loading

graphite-app bot commented Dec 20, 2024 •

edited

Loading

vercel bot commented Dec 20, 2024 •

edited

Loading

socket-security bot commented Dec 20, 2024 •

edited

Loading

hbjORbj commented Dec 20, 2024

emrysal Dec 26, 2024

emrysal Dec 26, 2024

CLAassistant commented Dec 26, 2024 •

edited

Loading

emrysal left a comment

github-actions bot commented Dec 26, 2024 •

edited

Loading

	customer.emailVerified === undefined ? "No" : customer.emailVerified ? "Yes" : "No",
	customer.emailVerified ? "Yes" : "No",

feat: Plain chat #18284

feat: Plain chat #18284

Conversation

nizzyabi commented Dec 20, 2024 • edited Loading

What does this PR do?

Mandatory Tasks (DO NOT REMOVE)

How should this be tested?

Checklist

github-actions bot commented Dec 20, 2024 • edited Loading

graphite-app bot commented Dec 20, 2024 • edited Loading

Graphite Automations

vercel bot commented Dec 20, 2024 • edited Loading

socket-security bot commented Dec 20, 2024 • edited Loading

hbjORbj commented Dec 20, 2024

emrysal Dec 26, 2024

Choose a reason for hiding this comment

emrysal Dec 26, 2024

Choose a reason for hiding this comment

CLAassistant commented Dec 26, 2024 • edited Loading

emrysal left a comment

Choose a reason for hiding this comment

github-actions bot commented Dec 26, 2024 • edited Loading

E2E results are ready!

nizzyabi commented Dec 20, 2024 •

edited

Loading

github-actions bot commented Dec 20, 2024 •

edited

Loading

graphite-app bot commented Dec 20, 2024 •

edited

Loading

vercel bot commented Dec 20, 2024 •

edited

Loading

socket-security bot commented Dec 20, 2024 •

edited

Loading

CLAassistant commented Dec 26, 2024 •

edited

Loading

github-actions bot commented Dec 26, 2024 •

edited

Loading