Skip to content

[Chore] Remove unused logo field from directory entries#28901

Merged
kodster28 merged 1 commit intoproductionfrom
directory-yaml-cleanup
Mar 10, 2026
Merged

[Chore] Remove unused logo field from directory entries#28901
kodster28 merged 1 commit intoproductionfrom
directory-yaml-cleanup

Conversation

@kodster28
Copy link
Collaborator

cleanup unused logo field in directory

@kodster28 kodster28 requested a review from a team as a code owner March 10, 2026 19:47
@github-actions
Copy link
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
* @cloudflare/pcx-technical-writing

@mvvmm
Copy link
Contributor

mvvmm commented Mar 10, 2026

/bonk is this logo field used anywhere?

Copy link
Contributor

@ask-bonk ask-bonk bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found. All 26 files cleanly remove the unused logo field (inline SVGs not referenced by any schema, component, or page). No logo fields remain in the directory after this PR.

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 10, 2026

Approved PR #28901.

Summary: Clean, straightforward cleanup. The logo field (containing inline SVGs) was present in 26 directory YAML files but was never defined in any schema, read by any component, or rendered on any page. Product icons are served via virtual:astro-icon from src/icons/, making these fields entirely dead data. After this PR, no logo fields remain in the directory collection.

github run

@kodster28 kodster28 enabled auto-merge (squash) March 10, 2026 19:52
@github-actions
Copy link
Contributor

@kodster28 kodster28 merged commit 2f0c9f1 into production Mar 10, 2026
15 checks passed
@kodster28 kodster28 deleted the directory-yaml-cleanup branch March 10, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants