Skip to content

production fix pack#39

Merged
kguzek merged 5 commits intomainfrom
fix/prod-preps
Sep 16, 2025
Merged

production fix pack#39
kguzek merged 5 commits intomainfrom
fix/prod-preps

Conversation

@kguzek
Copy link
Copy Markdown
Member

@kguzek kguzek commented Sep 12, 2025

resolves #19
resolves #37

@kguzek kguzek self-assigned this Sep 12, 2025
Copilot AI review requested due to automatic review settings September 12, 2025 23:01
@kguzek kguzek added the enhancement New feature or request label Sep 12, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a dead link in the hero section by refactoring navigation functionality and redirecting from a broken /about route to a fragment on the current page.

  • Extracted duplicate scroll navigation logic into a reusable helper function
  • Updated hero section button to navigate to an about fragment instead of a broken /about page
  • Applied the new helper function to existing navbar link functionality

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/lib/helpers.ts New helper function for smooth scrolling to page elements with URL updates
src/components/navigation/navbar-link.tsx Refactored to use the new helper function instead of duplicate code
src/app/page.tsx Updated hero button to use fragment navigation instead of broken /about link

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@pull-request-size pull-request-size bot added size/L and removed size/S labels Sep 12, 2025
@kguzek kguzek changed the title fix: dead link in hero section (#37) production fix pack Sep 12, 2025
@kguzek kguzek merged commit a12cab0 into main Sep 16, 2025
4 checks passed
@kguzek kguzek deleted the fix/prod-preps branch September 16, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

przycisk "dowiedz się wiecej" profilaktyka content

2 participants