fix: nameless in sidebar for orgs#10318
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Thank you for following the naming conventions! 🙏 |
86946c3 to
3077daa
Compare
nameless in sidebar for orgsnameless in sidebar for orgs
3077daa to
0236cdf
Compare
| */ | ||
| export function useOrgBranding() { | ||
| const orgBrandingContext = useContext(OrganizationBrandingContext); | ||
| if (orgBrandingContext === null) |
There was a problem hiding this comment.
The value can itself be null if the user isn't an org member. So, make a wrapper object that has orgBrand and that wrapper objet can be checked if it's nullish or not.
0236cdf to
1fe72e2
Compare
📦 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! 🙌 |
|
🤖 Meticulous spotted visual differences in 36 of 149 screens tested: view and approve differences detected. Last updated for commit a5f0731. This comment will update as new commits are pushed. |
Current Playwright Test Results Summary✅ 93 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 07/24/2023 10:25:26am UTC) Run DetailsRunning Workflow PR Update on Github Actions Commit: a5f0731 Started: 07/24/2023 10:23:16am UTC
|
| Test Case | Last 7 days Failures | Last 7 days Flakes |
|---|---|---|
|
Popup Tests should be able to reschedule
Retry 2 • Retry 1 • Initial Attempt |
1.80% (3)3 / 167 runsfailed over last 7 days |
97.01% (162)162 / 167 runsflaked over last 7 days |
sean-brydon
left a comment
There was a problem hiding this comment.
Tested -> Works great!
Nice work on fetching from provider instread of trpc each time we need it. We could likely do this across the app for a few things!
Co-authored-by: sean-brydon <55134778+sean-brydon@users.noreply.github.com>
What does this PR do?
useOrgBrandingValueswithuseOrgBrandingwhich gets the value from Provider. This would avoid unnecessary refetch of getBranding data which doesn't change frequently.Fixes #10310
Type of change
How should this be tested?
Mandatory Tasks
Checklist