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 a 404 page #159

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add a 404 page #159

wants to merge 4 commits into from

Conversation

NolwennP
Copy link
Contributor

No description provided.

Copy link
Contributor

@damien-carcel damien-carcel left a comment

Choose a reason for hiding this comment

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

Minor comments on end of files, then GTM

<div class="container not-found-page">
<div class="row">
<div class="col-xs-12">
<h1>Oho! What a strange place!</h1>
Copy link
Contributor

Choose a reason for hiding this comment

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

😆

@import "landing-page.less";
@import "404.less";
Copy link
Contributor

Choose a reason for hiding this comment

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

Please configure your editor to automatically add EOF 😉

styles/404.less Outdated
margin: 10px 0;
}
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Useless line

@meganelepalud-akeneo
Copy link
Contributor

Hello! :) Can we merge this?

@NolwennP
Copy link
Contributor Author

NolwennP commented Mar 24, 2021

We can. But, in order to work, it will also require adding, on the doc server, a redirection to this page whenever someone hits a not existing page.
And, also we will need to check whether the page still looks ok, as this PR was created a long long time ago.

@LevFlavien LevFlavien self-assigned this May 17, 2022
@LevFlavien LevFlavien removed their assignment Feb 1, 2023
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.

5 participants