fix(seo): wire branding.socialImageUrl into the OG/Twitter image fallback
Some checks failed
Architecture Governance / architecture (push) Has been cancelled
Some checks failed
Architecture Governance / architecture (push) Has been cancelled
Added in an earlier commit this session (branding OG image + gallery field), but SeoService.resetToDefaults() never actually read it - defaultImage fell back straight to appIconUrl/logoUrl, so the field the editor calls 'Social Share Image' had no runtime effect. Now it's checked first. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -97,7 +97,7 @@ export class SeoService {
|
|||||||
const defaultTitle = seoDefault?.title || `${this.siteName} — Marketplace`;
|
const defaultTitle = seoDefault?.title || `${this.siteName} — Marketplace`;
|
||||||
const defaultDescription = seoDefault?.description
|
const defaultDescription = seoDefault?.description
|
||||||
|| 'Modern marketplace for buying digital goods. Wide selection, convenient search, fast delivery.';
|
|| 'Modern marketplace for buying digital goods. Wide selection, convenient search, fast delivery.';
|
||||||
const defaultImage = this.resolveUrl(branding?.appIconUrl || branding?.logoUrl || '')
|
const defaultImage = this.resolveUrl(branding?.socialImageUrl || branding?.appIconUrl || branding?.logoUrl || '')
|
||||||
|| `${this.siteUrl}/icons/icon-512x512.png`;
|
|| `${this.siteUrl}/icons/icon-512x512.png`;
|
||||||
const canonicalUrl = seoDefault?.canonicalUrl ? this.resolveUrl(seoDefault.canonicalUrl) : this.siteUrl;
|
const canonicalUrl = seoDefault?.canonicalUrl ? this.resolveUrl(seoDefault.canonicalUrl) : this.siteUrl;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user