search

prezly / theme-nextjs-bea

News site built with NextJS, Typescript and Prezly SDK
https://theme-nextjs-bea-the-good-newsroom.vercel.app
MIT License
43 stars 11 forks source link

[CARE-5028] Fix - Parse preview search params in DynamicPreviewBranding #1148

Closed kudlajz closed 3 months ago

kudlajz commented 3 months ago

So, the problem here was that I was re-using the useThemeSettingsWithPreview hook which uses useThemeSettings which relies on context from ThemeSettings, but because DynamicPreviewBranding is rendered outside of ThemeSettingsProvider, it was injecting the default theme settings instead of the newsroom one's.

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
theme-nextjs-bea-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 16, 2024 4:25pm
3 Ignored Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **theme-nextjs-bea-cookbook** | ⬜️ Ignored ([Inspect](https://vercel.com/prezly/theme-nextjs-bea-cookbook/BRdco3v1WHPptvHg6jS1xJGN91qe)) | [Visit Preview](https://theme-nextjs-bea-cookbook-git-feature-care-5028-61a305-prezly1.vercel.app) | | May 16, 2024 4:25pm | | **theme-nextjs-bea-photography** | ⬜️ Ignored ([Inspect](https://vercel.com/prezly/theme-nextjs-bea-photography/4UouuZjK1TSdCuqNNxjLtSxZ4Uvv)) | | | May 16, 2024 4:25pm | | **theme-nextjs-bea-the-good-newsroom** | ⬜️ Ignored ([Inspect](https://vercel.com/prezly/theme-nextjs-bea-the-good-newsroom/4sJBjGePtmUpimiCZd4WfxLvPUsJ)) | | | May 16, 2024 4:25pm |