cal.com
cal.com copied to clipboard
merge /team layout with user layout
What does this PR do?
Fixes #4316
Environment: Staging(main branch) / Production
Loom video: https://www.loom.com/share/f7bef9fb34964ab1828990fa8e29636e
Type of change
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] Chore (refactoring code, technical debt, workflow improvements)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] This change requires a documentation update
@gitstart is attempting to deploy a commit to the cal Team on Vercel.
A member of the Team first needs to authorize it.
- Added name and username props to HeadSeoProps
- Imported PublicPage from pages/[user] instead of components/team/screens/Team
- Removed useTheme() hook call in TeamPage function, replaced with user?.theme || "light" as argument for useTheme(...) hook call (useMeQuery is used)
- Replaced getPlaceholderAvatar(...), CAL_URL + "/" + ... with avatar: CAL_URL + "/" + ... inside team object returned by getServerSideProps async function
@JeroenReumkens can you investigate if this is useful for the new booking page refactor? if yes, please approve, if no, close.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Updated |
---|---|---|---|
cal | ✅ Ready (Inspect) | Visit Preview | Dec 2, 2022 at 10:16AM (UTC) |
Hey @gitstart Thanks a lot for this contribution! 🙏 I'm currently working in rewriting the full layout of the booking pages, and I think it will be very hard to merge these two changes. I will however look into your changes and take in any changes that are valuable to have, so your work isn't wasted. Hope you understand 🫶
— Jeroen.