Skip to content

Conversation

alekskjo
Copy link
Member

This PR replaces #43.

The original PR was based on previously stacked feature branches. After those branches were merged, GitHub continued to include the earlier work in the comparison, which prevented the code owner approval requirement from being satisfied correctly.

This branch was created from the latest main and contains only the intended shared header and navigation changes.

Verified with:

  • npm ci
  • npm run format:check
  • npm run lint
  • npm run build

@alekskjo alekskjo requested a review from a team as a code owner October 10, 2026 08:22
@erikbsa erikbsa self-assigned this Oct 10, 2026
Copy link
Member

@erikbsa erikbsa left a comment

Choose a reason for hiding this comment

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

Looks good to me. The shared header and navigation make the pages more consistent, and the changes look ready to merge. Approved.

@erikbsa erikbsa merged commit 8816fb5 into main Oct 10, 2026
@erikbsa erikbsa deleted the feat/shared-site-header-clean branch October 10, 2026 11:22
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants