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

feat: Add custom fallback to handle 404 errors on azure #104

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

OLILHR
Copy link
Member

@OLILHR OLILHR commented Nov 5, 2024

on a scale from cringe to 10? @hf-kklein @DeltaDaniel @emmastoeck

image

@OLILHR OLILHR linked an issue Nov 5, 2024 that may be closed by this pull request
4 tasks
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
@Hochfrequenz Hochfrequenz deleted a comment from github-actions bot Nov 5, 2024
Copy link

github-actions bot commented Nov 5, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-104.westeurope.5.azurestaticapps.net

@OLILHR OLILHR marked this pull request as ready for review November 5, 2024 20:13
Copy link

github-actions bot commented Nov 5, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-104.westeurope.5.azurestaticapps.net

@hf-kklein
Copy link
Contributor

Ich würde schon dazuschreiben, was wir wissen: z.b.

  1. es gibt das EBD
  2. aber es ist irgendwie anders gestrickt als die anderen. Sonst könnte man das nicht/falsch verstehen.

Und dann für mich gern ein bisschen deutlicher, dass wir aktuell nicht alles können aber unsere User einladen beizutragen und ihre contributions gerne entgegen nehmen.

@OLILHR
Copy link
Member Author

OLILHR commented Nov 6, 2024

Ich würde schon dazuschreiben, was wir wissen: z.b.

  1. es gibt das EBD
  2. aber es ist irgendwie anders gestrickt als die anderen. Sonst könnte man das nicht/falsch verstehen.

Und dann für mich gern ein bisschen deutlicher, dass wir aktuell nicht alles können aber unsere User einladen beizutragen und ihre contributions gerne entgegen nehmen.

die Sache ist, dass User nicht auf "fehlende EBDs" stoßen können, weil das Input-Select für die EBD-Suche seine Optionen aus den *.svg Files vom Submodule sammelt. Im Input ist zwar ein smart filter eingebaut, der nach matches sucht, aber der User kann nicht enter drücken und bekommt 404.

Auf der Error-Page würde der User nur landen, wenn er auf die Idee kommt, das EBD "E_9999", von welchem er glaube/hofft, dass es exisiert, manuell in die URL einzugeben /ebd/FV2404/E_9999.

@OLILHR
Copy link
Member Author

OLILHR commented Nov 6, 2024

image

geht das in eine bessere Richtung?

Copy link

github-actions bot commented Nov 6, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-104.westeurope.5.azurestaticapps.net

@emmastoeck
Copy link

Haben wir eine Farbe mit etwas höherem Kontrast? Ich finde die Farbe etwas zu hell, um es leicht lesen zu können.
Die Formulierung finde ich süß :D

Copy link

github-actions bot commented Nov 6, 2024

Azure Static Web Apps: Your stage site is ready! Visit it here: https://white-water-047c71203-104.westeurope.5.azurestaticapps.net

@OLILHR
Copy link
Member Author

OLILHR commented Nov 6, 2024

Haben wir eine Farbe mit etwas höherem Kontrast? Ich finde die Farbe etwas zu hell, um es leicht lesen zu können. Die Formulierung finde ich süß :D

image

habe die font-weight noch etwas erhöht.

ui/accessbility (und Text) muss auf jeden Fall noch mal später überarbeitet werden; das ist alles erst mal nur ein erster Draft, um die Basics und Features einzubauen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

❌ 404 fallback
3 participants