Skip to content

fix: show real per-project last-edited dates and add sort control#20

Merged
yepzdk merged 2 commits intomainfrom
feature/fix-project-last-edited-dates
Apr 29, 2026
Merged

fix: show real per-project last-edited dates and add sort control#20
yepzdk merged 2 commits intomainfrom
feature/fix-project-last-edited-dates

Conversation

@yepzdk
Copy link
Copy Markdown
Contributor

@yepzdk yepzdk commented Apr 29, 2026

Summary

  • CI runners used a shallow clone, so the projectDates data loader couldn't read per-project commit history and every home-page card rendered the same timestamp on the deployed site. Both workflows now check out with fetch-depth: 0.
  • Added a sort control to HomeFeatures.vue: Last edited ↓ / ↑ and Alphabetically ↓ / ↑. Default is newest-first by last edited; projects without a known timestamp sort last; alphabetical sort uses Danish collation and strips HTML from titles.
  • CHANGELOG updated.

Test plan

  • Local build succeeds (npx vitepress build docs)
  • After merge, home page on https://itk-dev.github.io/research-projects/ shows distinct dates per project
  • Sort control reorders cards correctly for all four options
  • Lønforhandlingssystem (with the lock emoji) sorts correctly alphabetically
  • Verify_build workflow on this PR passes

Generated with Claude Code

yepzdk and others added 2 commits April 29, 2026 13:36
The CI runner used a shallow clone, so the projectDates data loader
fell back to the same commit timestamp for every project card on the
deployed site. Fetch full history in both workflows and add a sort
control on the home page (last edited / alphabetically, asc/desc).

Co-authored-by: Claude <noreply@anthropic.com>
Lønforhandlingssystem was sorted by its decorative 🔒 prefix instead of
the actual title. Strip leading non-letter characters from the sort key
so it lands between Deltag and Opkrævningsoverblik as expected.

Co-authored-by: Claude <noreply@anthropic.com>
@yepzdk yepzdk merged commit d6434d2 into main Apr 29, 2026
1 check passed
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.

1 participant