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

frontend: Added error Page #35

Closed
wants to merge 0 commits into from

Conversation

PrathamX595
Copy link
Contributor

Added Error Page

Fixes: #21

Description

added an error page which is rendered when an unknown route is accessed, it contains a link to go back to home page of the game.

How to Test

  1. use the command npm run dev in the console and open the application.
  2. In the search box add /k (or any other route which is not recognized by the router) after the existing text.

Checklist

  • I have tested these changes locally.
  • I have reviewed the code and ensured it follows the project's coding guidelines.
  • I have updated the documentation, if necessary.
  • I have assigned reviewers to this pull request.

Screenshots (if applicable)

70e10a13-76c8-4561-8532-b4486e8272bd

@PrathamX595 PrathamX595 marked this pull request as draft June 1, 2024 09:36
@PrathamX595 PrathamX595 marked this pull request as ready for review June 1, 2024 09:36
@kuv2707
Copy link
Collaborator

kuv2707 commented Jun 1, 2024

@PrathamX595 Please remove the merge commit and use rebase to sync your branch with upstream/master.

Also please adhere to the commit message format as described in CONVENTIONS.md

@PrathamX595 PrathamX595 marked this pull request as draft June 1, 2024 11:05
@kuv2707 kuv2707 added the area: Frontend (structure) Issues related to the react frontend label Jun 1, 2024
@PrathamX595
Copy link
Contributor Author

@kuv2707 I am having trouble removing the merge commit can someone assist me ?

@PrathamX595 PrathamX595 marked this pull request as ready for review June 2, 2024 15:54
@PrathamX595 PrathamX595 force-pushed the master branch 2 times, most recently from d0d1b15 to 8aa2dfe Compare June 2, 2024 16:34
@PrathamX595
Copy link
Contributor Author

PrathamX595 commented Jun 2, 2024

@kuv2707 I have implemented the recommended changes let me know if anything else needs to be done.

@@ -27,6 +27,10 @@ type EventResult = {
message: string;
};

declare global {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove all changes to this file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved: Shivansh area: Frontend (structure) Issues related to the react frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Design an error page
3 participants