Feat(not-found): not found page #50

Merged
jakob.scheid merged 22 commits from feature/not-found-page into main 2026-05-14 16:21:30 +02:00
Member

Related issues

Closes: #48

# Related issues Closes: #48
johannes.vos added this to the 0.1.0 milestone 2026-05-14 12:38:54 +02:00
johannes.vos added the
Priority
High
2
Kind/FeatureArea/Global
labels 2026-05-14 12:38:54 +02:00
johannes.vos self-assigned this 2026-05-14 12:38:54 +02:00
johannes.vos added this to the Project Board project 2026-05-14 12:38:54 +02:00
johannes.vos added 15 commits 2026-05-14 12:38:54 +02:00
jakob.scheid moved this to In progress in Project Board on 2026-05-14 13:08:40 +02:00
jakob.scheid moved this to Review in Project Board on 2026-05-14 13:08:43 +02:00
jakob.scheid reviewed 2026-05-14 13:15:27 +02:00
@@ -39,0 +39,4 @@
},
{
path: '/:pathMatch(.*)*',
name: 'not-found',
Owner

Use camel case (notFound).

Use camel case (*notFound*).
jakob.scheid marked this conversation as resolved
jakob.scheid reviewed 2026-05-14 13:17:16 +02:00
@@ -39,3 +45,3 @@
];
const router = createRouter({
const router = createRouter({
Owner

Omit the space at the end of the line.

Omit the space at the end of the line.
jakob.scheid marked this conversation as resolved
jakob.scheid reviewed 2026-05-14 13:18:20 +02:00
@@ -0,0 +11,4 @@
</template>
<style scoped>
.global-header {
Owner

Copying styles is a very bad idea. Nothing should be redundant.

Copying styles is a very bad idea. Nothing should be redundant.
jakob.scheid marked this conversation as resolved
jakob.scheid reviewed 2026-05-14 13:19:17 +02:00
@@ -0,0 +3,4 @@
<header class="global-header">
<h1>Seekra</h1>
<span class="error-message">
ERROR 404: Page Not Found
Owner

Do not write ERROR uppercase.

Do not write *ERROR* uppercase.
jakob.scheid marked this conversation as resolved
jakob.scheid reviewed 2026-05-14 13:20:28 +02:00
@@ -0,0 +1,51 @@
<template>
<div class="search-content">
Owner

Do not use a container .search-content in a not found page.

Do not use a container *.search-content* in a not found page.
jakob.scheid marked this conversation as resolved
jakob.scheid reviewed 2026-05-14 13:21:22 +02:00
@@ -0,0 +2,4 @@
<div class="search-content">
<header class="global-header">
<h1>Seekra</h1>
<span class="error-message">
Owner

No HTTP status code. That is confusing for people who are not familiar with technology.

No HTTP status code. That is confusing for people who are not familiar with technology.
jakob.scheid marked this conversation as resolved
jakob.scheid reviewed 2026-05-14 13:22:08 +02:00
@@ -0,0 +1,51 @@
<template>
<div class="search-content">
<header class="global-header">
Owner

The not found page should be minimal, so omit the header. The navigation bar already contains Seekra.

The not found page should be minimal, so omit the header. The navigation bar already contains *Seekra*.
jakob.scheid marked this conversation as resolved
jakob.scheid changed title from Add NotFound.vue to Add NotFound.vue #48 2026-05-14 13:26:25 +02:00
johannes.vos added 7 commits 2026-05-14 16:06:52 +02:00
jakob.scheid approved these changes 2026-05-14 16:21:16 +02:00
jakob.scheid left a comment
Owner

Good

Good
jakob.scheid merged commit fc23a6d185 into main 2026-05-14 16:21:30 +02:00
jakob.scheid deleted branch feature/not-found-page 2026-05-14 16:21:30 +02:00
jakob.scheid moved this to Done in Project Board on 2026-05-14 16:21:36 +02:00
jakob.scheid changed title from Add NotFound.vue #48 to Feat: not found page 2026-07-20 12:01:25 +02:00
jakob.scheid changed title from Feat: not found page to Feat(not-found): not found page 2026-07-20 12:22:07 +02:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Seekra/frontend#50