-
Notifications
You must be signed in to change notification settings - Fork 6.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: cal video issues #12546
fix: cal video issues #12546
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
5 Ignored Deployments
|
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link to collect XP and win prizes! |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
No failed tests 🎉 |
@@ -214,7 +214,7 @@ export function VideoMeetingInfo(props: VideoMeetingInfo) { | |||
<h3>{t("invitee_timezone")}:</h3> | |||
<p>{booking.user?.timeZone}</p> | |||
<h3>{t("when")}:</h3> | |||
<p> | |||
<p suppressHydrationWarning={true}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apps/web/server.js
Outdated
const { createServer } = require("https"); | ||
const { parse } = require("url"); | ||
const next = require("next"); | ||
const fs = require("fs"); | ||
const port = 3000; | ||
const dev = process.env.NODE_ENV !== "production"; | ||
const app = next({ dev }); | ||
const handle = app.getRequestHandler(); | ||
|
||
const httpsOptions = { | ||
key: fs.readFileSync("./app.cal.local-key.pem"), | ||
cert: fs.readFileSync("./app.cal.local.pem"), | ||
}; | ||
|
||
app.prepare().then(() => { | ||
createServer(httpsOptions, async (req, res) => { | ||
const parsedUrl = parse(req.url, true); | ||
await handle(req, res, parsedUrl); | ||
}).listen(port, (err) => { | ||
if (err) throw err; | ||
console.log(`ready - started server on url: https://app.cal.local:${port}`); | ||
}); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This script is for serving nextjs server over https for testing organization's cal video meeting links
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, need @hariombalhara for final review for org changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, need @hariombalhara for final review for org changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep this PR with only the proposed fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code now LGTM. Waiting for checks
Co-authored-by: Joe Au-Yeung <65426560+joeauyeung@users.noreply.github.com>
Co-authored-by: Joe Au-Yeung <65426560+joeauyeung@users.noreply.github.com>
Fixes: #11136
Before:-
After:-