Skip to content

seo: add policy and blog page descriptions - #64

Open
FenjuFu wants to merge 4 commits into
mainfrom
seo-url-driven-filters
Open

seo: add policy and blog page descriptions#64
FenjuFu wants to merge 4 commits into
mainfrom
seo-url-driven-filters

Conversation

@FenjuFu

@FenjuFu FenjuFu commented Jul 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Summary

Current main already contains the redesigned Projects and Events directories from #100, including URL-synchronized filters and page-specific descriptions. This PR now preserves that newer implementation and narrows its remaining diff to SEO metadata that main still lacks:

  • add a page-specific description to blog tag archives;
  • allow MarkdownLayout frontmatter to provide description and descriptionEn, and emit both in the page metadata;
  • add bilingual metadata for the CLA, security policy, and community values pages;
  • update the changelog for the narrowed scope.

The obsolete Projects, Events, Adopters, Contribute, blog index/category, and deleted Landscape implementations from the old branch were not carried over.

Review follow-up

The requested descriptionEn field is present on the CLA page. MarkdownLayout now also consumes it, combining description and descriptionEn in the emitted meta description.

Validation

  • npm run check:astro: 114 files, 0 errors, 0 warnings, 0 hints
  • focused ESLint on all changed Astro files: pass
  • focused Prettier check on all six changed files: pass
  • npm exec -- astro build: 71 pages built successfully

The full npm run check reaches ESLint but is blocked on Windows by an unrelated parser/CRLF error in the unchanged main file src/components/common/Image.astro. The changed files pass focused ESLint.

@vercel

vercel Bot commented Jul 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
website Ready Ready Preview Sep 1, 2026 6:13am UTC

dongjiang1989
dongjiang1989 reviewed Jul 24, 2026
View reviewed changes

@dongjiang1989 dongjiang1989 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.

Thanks @FenjuFu
Great feature.

Need add Changelog to CHANGELOG.md

@FenjuFu

FenjuFu commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

@dongjiang1989 Thanks for the review. Added the SEO and URL-driven filtering changes to the CHANGELOG.md Unreleased section in commit 8570bb1.

@dongjiang1989

dongjiang1989 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Please fix CI fail. DCO setting

dongjiang1989
dongjiang1989 requested changes Jul 24, 2026
View reviewed changes

@dongjiang1989 dongjiang1989 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.

Need to check all added English and Chinese descriptions

Comment thread src/pages/cla.astro Outdated
@FenjuFu

FenjuFu commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

Thanks @dongjiang1989! Added the changelog entry under [Unreleased] → Changed in CHANGELOG.md (commit 8570bb1), already pushed to this branch. Ready for another look.

@dongjiang1989

dongjiang1989 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

@FenjuFu Need fix DCO sign

@FenjuFu
FenjuFu force-pushed the seo-url-driven-filters branch from 8570bb1 to 05dc87e Compare July 28, 2026 10:10
@dongjiang1989

dongjiang1989 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

@FenjuFu need rebase and fix review commit

FenjuFu added 2 commits August 5, 2026 10:28
@FenjuFu
seo: URL-driven list filters and per-page meta descriptions
a6c5075 
Make filtered views of the projects and events pages crawlable and
shareable by moving filter state into URL query parameters:

- /projects: category/language filters are now server-rendered links
  (?category=...&language=...) filtered on the server, replacing the
  client-side JS show/hide script; add an empty-state with a clear link
- /events: switch to server rendering with ?type=... filter links,
  replacing the client-side filter script; event status (upcoming/
  ongoing/past) is now computed at request time instead of being frozen
  at build time

Canonical URLs already strip query strings, so filtered views
canonicalize to the base list page and add no duplicate content.

Also add unique meta descriptions to pages that previously fell back to
the site-wide default: projects, events, adopters, landscape,
contribute, CLA, values, security and the blog list/category/tag pages
(MarkdownLayout now passes description through).

Signed-off-by: FenjuFu <fufenjupku@gmail.com>
@FenjuFu
docs: update changelog for SEO filters
9cf373b 
Signed-off-by: FenjuFu <fufenjupku@gmail.com>
@FenjuFu
FenjuFu force-pushed the seo-url-driven-filters branch from 05dc87e to 9cf373b Compare August 5, 2026 02:31
@FenjuFu

FenjuFu commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

@dongjiang1989 Done — rebased onto latest main and resolved the CHANGELOG.md conflict (the SEO filters entry now lives under the existing [Unreleased] → Changed section alongside the Astro 7 upgrade notes). Both commits are signed off; astro check passes (103 files, 0 errors). Branch is now mergeable. PTAL.

dongjiang1989
dongjiang1989 approved these changes Aug 5, 2026
View reviewed changes

@dongjiang1989 dongjiang1989 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.

Thanks

@FenjuFu
fix(cla): add English metadata description
7060d20 
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
@FenjuFu
merge(main): preserve page-specific SEO metadata
eb45e56 
Signed-off-by: FenjuFu <92919259+FenjuFu@users.noreply.github.com>
@pull-request-size pull-request-size Bot added size/M and removed size/L labels Sep 1, 2026
@FenjuFu FenjuFu changed the title seo: URL-driven list filters and per-page meta descriptions seo: add policy and blog page descriptions Sep 1, 2026
@FenjuFu

FenjuFu commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

The remaining CLA descriptionEn thread is addressed on current head eb45e56 and is now resolved: the frontmatter includes the English description, and MarkdownLayout consumes both language fields. The branch is mergeable and every current check is green, including build, link check, CodeQL, DCO, CLA, and Vercel.

@dongjiang1989, could you please take a final look when convenient?

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

Reviewers

@dongjiang1989 dongjiang1989 Awaiting requested review from dongjiang1989

Assignees

No one assigned

Labels

size/M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@FenjuFu @dongjiang1989