Skip to content

Hotfix/minor fixes#68

Merged
andres15alvarez merged 13 commits into
devfrom
hotfix/minor-fixes
May 26, 2025
Merged

Hotfix/minor fixes#68
andres15alvarez merged 13 commits into
devfrom
hotfix/minor-fixes

Conversation

@andres15alvarez
Copy link
Copy Markdown
Member

This pull request introduces several updates across multiple components to enhance functionality, improve user experience, and simplify the codebase. Key changes include updates to the layout and navigation behavior, improvements to product and order handling logic, and refactoring of search and filtering functionality.

Layout and Navigation Updates:

  • Made the navigation bar sticky by adding sticky top-0 classes to the NavBar container in src/app/(shop)/layout.tsx.

Product and Order Handling Improvements:

Search and Filtering Refactor:

  • Refactored SearchPage to simplify filter handling, remove redundant state, and improve product fetching logic. Breadcrumb generation and filter application have been streamlined. [1] [2] [3] [4]

Minor Adjustments and Bug Fixes:

  • Ensured useEffect in Home sets an empty array for recommended products when the user is not authenticated.
  • Removed unnecessary window.location.reload() in RegisterForm and replaced it with a direct navigation to the home page.

Codebase Simplifications:

  • Updated Avatar to clear the cart on logout and added a safe handler for profile navigation. [1] [2] [3] [4]
  • Cleaned up unused imports and removed redundant state in Navbar. [1] [2]

@andres15alvarez andres15alvarez self-assigned this May 25, 2025
@andres15alvarez andres15alvarez added the bug Something isn't working label May 25, 2025
@andres15alvarez andres15alvarez merged commit 190689f into dev May 26, 2025
1 check passed
@andres15alvarez andres15alvarez deleted the hotfix/minor-fixes branch May 26, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant