Skip to content

feat: bring back last-updated metadata and add filtered git history#478

Open
rudigier wants to merge 15 commits into
mainfrom
feature/page-meta-recent-changes
Open

feat: bring back last-updated metadata and add filtered git history#478
rudigier wants to merge 15 commits into
mainfrom
feature/page-meta-recent-changes

Conversation

@rudigier
Copy link
Copy Markdown
Collaborator

@rudigier rudigier commented Apr 3, 2026

What changed

  • bring back Last updated metadata (date + author) for each page.
  • Added expandable commit history for each page.
  • Linked commit messages directly to their GitHub commit pages.
  • Filtered out non-relevant commits via an exclusion list.
  • Limited visible history to relevant commits from the last 5 years.
  • Centralised repository URL in config.js (repoUrl).
  • Refactored git history logic out of gulp/html.js into gulp/helpers/git-metadata.js.

Why

  • Make page maintenance history transparent and useful for contributors.
  • Reduce noise by hiding technical/maintenance-only commits.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 3, 2026

Deploy Preview for accessibility-developer-guide ready!

Name Link
🔨 Latest commit a4c8d82
🔍 Latest deploy log https://app.netlify.com/projects/accessibility-developer-guide/deploys/6a06d360f2250000083ab58f
😎 Deploy Preview https://deploy-preview-478--accessibility-developer-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@rudigier rudigier requested a review from backflip April 3, 2026 21:43
@backflip
Copy link
Copy Markdown
Collaborator

backflip commented May 6, 2026

I like it!

The main challenge is probably filtering irrelevant commits. Example: https://deploy-preview-478--accessibility-developer-guide.netlify.app/knowledge/colours-and-contrast/ and https://deploy-preview-478--accessibility-developer-guide.netlify.app/knowledge/keyboard-only/ both show not too relevant commits in Show recent relevant commits for this page. I think I would remove that feature as the View full history on GitHub is probably helpful enough. WDYT?

@rudigier
Copy link
Copy Markdown
Collaborator Author

rudigier commented May 15, 2026

@backflip What I adjusted:

Most changes are not visible on normal guide pages they support developers and maintenance.

  • Reverted changes in page meta (removed author and history).
  • git and frontmatter helpers for updates, merges, and page lists.
  • New components / pages:
    • Page updates (public): Curated component that highlights the latest page updates. The configuration is stored in config/recent-pages.js.
    • Recent changes (developer view only): Shows which pages changed in each merge to main. Includes a simple configurator for updating config/recent-pages.js.
    • Page revisions (developer view only): Table of all pages with the date of the last meaningful update, helping identify pages that need a refresh.
  • Cleanup of unused code.

@rudigier
Copy link
Copy Markdown
Collaborator Author

rudigier commented May 15, 2026

Page updates
Screenshot 2026-05-15 at 10 18 51

Developer Menu
Screenshot 2026-05-15 at 10 18 03

Recent changes
Screenshot 2026-05-15 at 10 18 25

Page revisions
Screenshot 2026-05-15 at 10 18 40

@backflip
Copy link
Copy Markdown
Collaborator

@rudigier, these look great! However, I'm not sure what problem we are trying to solve with Developer Menu, Recent changes and Page revisions. I'm somewhat afraid of introducing so much code to the build system to – unless I'm misunderstanding – rebuild parts of GitHub's history UI within our porject.

@rudigier
Copy link
Copy Markdown
Collaborator Author

rudigier commented May 22, 2026

@backflip Fair point! We’re not rebuilding (the complete) GitHub history ;)

The idea was to make our work on the guide visible, show that ADG is actively maintained, and give maintainers an overview of content that may need a refresh soon because it’s potentially outdated. If you have a better idea or a less code involved solution, let me know.

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.

2 participants