Skip to content

Comments

Feat/error & logout pages#22

Merged
CMolinaBetancourt merged 4 commits intomainfrom
feat/error&logout
Nov 8, 2025
Merged

Feat/error & logout pages#22
CMolinaBetancourt merged 4 commits intomainfrom
feat/error&logout

Conversation

@CMolinaBetancourt
Copy link
Collaborator

🎯Type of Change (Select one or more):

  • Feat ✨: New feature or functionality.

This PR...

Added error and logout pages.


📝 What I Did (Detailed Work):

  • Created the Pages: Create the error/not found page and the logout page.
  • Adjusted App and Main, so these new pages are visible.
  • Validated that the Navbar and Footer are visible on them.

🧪 How to Test (Steps to Verify):

  1. Navigate to the preview deploy.
  2. Review the functionality of the not found page on a non-existent route.
  3. Log in, and perform a log out to validate the functionality of the logout page.

🤔 What I learned (gotchas):

Adjust the routes based on the defined architecture.

@netlify
Copy link

netlify bot commented Nov 5, 2025

Deploy Preview for mint-chip ready!

Name Link
🔨 Latest commit d6a3d6d
🔍 Latest deploy log https://app.netlify.com/projects/mint-chip/deploys/690bad232c42820008226049
😎 Deploy Preview https://deploy-preview-22--mint-chip.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link
Collaborator

@magali-la magali-la left a comment

Choose a reason for hiding this comment

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

works great! ready to merge. The Netlify preview doesn't allow to change the path to test the error page for some reason, so we can test the error page nav after merge

@CMolinaBetancourt CMolinaBetancourt merged commit eb1488d into main Nov 8, 2025
5 checks passed
@CMolinaBetancourt CMolinaBetancourt deleted the feat/error&logout branch November 10, 2025 22:39
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.

3 participants