Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Error404 Module and Modify App.jsx #28

Merged
merged 2 commits into from
Jan 28, 2025
Merged

Conversation

Swapnadip2005
Copy link
Contributor

Description

This pull request adds a new Error404 module and modifies the App.jsx and index.html files.

  • The Error404 module includes its JSX file, CSS module, and assets to improve error handling and user experience.
  • Updates to App.jsx ensure integration of the Error404 module.
  • Modifications to index.html enhance the application's structure and meta content.

Motivation:
These changes aim to improve the application's error handling by providing a dedicated Error404 page, ensuring users are guided effectively when encountering broken links or missing pages.


Type of Change

  • 🆕 New Feature
  • 🐛 Bug Fix
  • 🖼 Wallpaper Added
  • 🎨 UI/Design Update
  • 🔄 Refactor/Code Improvement
  • 📂 Other (Specify here):

Issue Linked: Fixes #20

Video

404Error.mp4

Checklist

  • 📖 I have read and followed the Contributing Guidelines.
  • ✅ My code adheres to the project's style guidelines.
  • 🧪 I have tested my changes locally and ensured no existing functionality is broken.

Additional Notes

The Error404 module is a new addition and may require future adjustments as the application evolves.

@Parnab03 Parnab03 self-requested a review January 28, 2025 14:18
Copy link
Owner

@Parnab03 Parnab03 left a comment

Choose a reason for hiding this comment

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

You’ve done a great job! We will wait for the other contributor to submit their PR for this issue within a day. If they don’t, we will accept yours. If they do, the best one will be selected.

@Parnab03 Parnab03 merged commit 90582aa into Parnab03:main Jan 28, 2025
1 check passed
@Parnab03
Copy link
Owner

Congratulations!

@Swapnadip2005
Copy link
Contributor Author

Swapnadip2005 commented Jan 28, 2025 via email

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.

🚀 404: PAGE NOT FOUND PAGE
2 participants