Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

Mv 317 add 404 page #150

Merged
merged 9 commits into from
Mar 7, 2023
Merged

Mv 317 add 404 page #150

merged 9 commits into from
Mar 7, 2023

Conversation

ostatni5
Copy link
Contributor

@ostatni5 ostatni5 commented Mar 2, 2023

  • Add 404 page in ui
  • Redirect all unknown paths to ui

image
image

@ostatni5 ostatni5 self-assigned this Mar 2, 2023
@ostatni5 ostatni5 force-pushed the MV-317-add-404-page branch 2 times, most recently from 72222ea to df666d4 Compare March 2, 2023 14:37
@ostatni5 ostatni5 marked this pull request as ready for review March 7, 2023 11:31
Comment on lines 29 to 34
inset: {
...generateFractionPercentages(20),
},
height: {
...generateFractionPercentages(10),
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need to generate this? Isn't the arbitrary values syntax enough in that case?

Copy link
Contributor Author

@ostatni5 ostatni5 Mar 7, 2023

Choose a reason for hiding this comment

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

You are right. Arbitrary values are good enough. I didn't notice that syntax when I was working on it.

@ostatni5 ostatni5 merged commit 198ef3b into master Mar 7, 2023
@ostatni5 ostatni5 deleted the MV-317-add-404-page branch March 7, 2023 15:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants