Skip to content

Error 404 Page Not Found#78

Merged
keiffer01 merged 4 commits into
masterfrom
404-error-page
Jul 27, 2020
Merged

Error 404 Page Not Found#78
keiffer01 merged 4 commits into
masterfrom
404-error-page

Conversation

@keiffer01
Copy link
Copy Markdown
Collaborator

@keiffer01 keiffer01 commented Jul 17, 2020

What is a quick description of the change?

Creates an Error 404 Page Not Found page if the user attempts to visit a page that does not exist.

Is this fixing an issue?

none

Are there more details that are relevant?

The ErrorPageNotFound component was added to the Errors directory, and is imported into the App component.

Check lists (check x in [ ] of list items)

  • [ ] Test written/updating
  • [ ] Tests passing
  • Coding style (indentation, etc)

Just renders a new page, no new logic was written.

Any additional comments?

GIF below (it is dark because I was using a Night Mode extension):
Untitled_ Jul 17, 2020 2_50 AM

@keiffer01 keiffer01 requested review from anan-ya-y and zghera July 17, 2020 07:14
@keiffer01 keiffer01 self-assigned this Jul 17, 2020
@keiffer01
Copy link
Copy Markdown
Collaborator Author

To keep consistent with PR #77, the ErrorNotFound directory was restructured as its own file in the Errors directory. The "Are there more details that are relevant?" section has been updated accordingly.

@keiffer01 keiffer01 marked this pull request as ready for review July 22, 2020 00:06
@keiffer01 keiffer01 merged commit 07b9a9e into master Jul 27, 2020
@keiffer01 keiffer01 deleted the 404-error-page branch July 27, 2020 05:35
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.

4 participants