Skip to content

🖌️ Clean up meta tags#2234

Closed
JFWooten4 wants to merge 20 commits into
stellar:mainfrom
JFWooten4:tent
Closed

🖌️ Clean up meta tags#2234
JFWooten4 wants to merge 20 commits into
stellar:mainfrom
JFWooten4:tent

Conversation

@JFWooten4

Copy link
Copy Markdown
Contributor

this should merge cleanly after #2231

it just polishes up some of the old SEO

prettier ignores tag files
* start with basic deduplcation

* subfolders to organize lab

* consistent SDP numbering imgs (#20)

* de-duplicate the new subfolders

* littler nitty folder categories

* fix isolated file names

* fix wierd file names

* simplify English funtion metas

* move out unused folder
a lot of root imgs are not used throughout

* rename 14.X loose files

* rename 20x loose file
Getting rid of this on the basis of a conversation affirming its duplication - 20 May 2025 with Bri.
Copilot AI review requested due to automatic review settings February 7, 2026 06:57
@JFWooten4 JFWooten4 marked this pull request as draft February 7, 2026 06:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

While an editor might need to change a display menu or slug, I don't think they will commonly adjust the sidebar positioning. That happens more in bulk, as far as I've seen—like in stellar#686 where none of the editorial content changes.
@JFWooten4

JFWooten4 commented Feb 11, 2026

Copy link
Copy Markdown
Contributor Author

It'd also be super if this could clean up some of the tangled API endpoints. There are areas with inconsistent and unreadable blobs for filenames. Some paths to start with:

  • docs\data\indexers\build-your-own\ingest-sdk\developer_guide\ledgerbackends\bufferedstoragebackend.mdx
    • Has underscores in folder, too.
    • I'm thinking kebab-case.
      • It's done proper in galexie\admin_guide\full-history-exporting.mdx.
  • docs\data\apis\rpc\api-reference\methods
    • Still uses capitalized filenames which are...
    • It works but is very standalone.
    • Produces a capitalized URL.
  • docs\data\apis\horizon\api-reference\resources\claimablebalances
    • The worst thing is that aggregations\order-books splits instead.
  • data\analytics\analytics-providers\analytics-providers.mdx

@ElliotFriend

Copy link
Copy Markdown
Contributor

Thanks for the effort here — 1,584 files is no small thing, and that's honestly part of why I'm closing it.

Having sampled the changeset, the bulk of it is: (a) reordering frontmatter keys within pages (no rendering or SEO effect, and since no linter enforces key order, it would drift back immediately), (b) converting reference-style markdown links to inline (a style preference we don't consider an upgrade — reference-style is a deliberate choice in several sections), and (c) ~780 files under i18n/, which is about to be removed entirely (see #2410 — the maintained-translations experiment is winding down), so half this PR evaporates regardless.

Set against that, a 1,584-file review isn't a good trade for either of us. If there are pages with genuinely duplicated or redundant meta/head blocks, I'd gladly take that as a small, targeted PR after #2410 lands — a focused diff of real removals will merge fast.

One more thing: the API-reference file-renaming idea you floated in the comments would break URLs and needs redirects, so please raise that as an issue for discussion before any code — it's not a decision to make inside a cleanup PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants