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

1 1 anlegen der seite impressum #3

Merged
merged 5 commits into from
Nov 8, 2024

Conversation

BSO-Jannis
Copy link
Contributor

Text embedded within the impressum-page is just used as example text from user story to file the page

@BSO-Jannis BSO-Jannis self-assigned this Nov 5, 2024
@BSO-Jannis BSO-Jannis linked an issue Nov 5, 2024 that may be closed by this pull request
Copy link
Contributor

@saschadoemer saschadoemer left a comment

Choose a reason for hiding this comment

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

In der Grundstruktur sind noch einige Dinge, die keinen Sinn mehr ergeben. D.h. beispielsweise die Tests der App oder auch das Setup der Tests. Die sollten auch entsprechend raus.

@saschadoemer
Copy link
Contributor

Der Build und die Integration der Dependencies wirft wirklich viele Warnings bzgl. bereits deaktivierter oder auch nicht mehr zu nutzender Packages. Das solltest du dir vorher anschauen.

image

@saschadoemer
Copy link
Contributor

In der Grundstruktur liegen auch noch die alten Logos, wie auch die CSS? Welche Daten werden da noch gebraucht?

@saschadoemer
Copy link
Contributor

Der public Ordner scheint auch noch ein Relikt der vorherigen Struktur zu sein.

@BSO-Jannis
Copy link
Contributor Author

Der Build und die Integration der Dependencies wirft wirklich viele Warnings bzgl. bereits deaktivierter oder auch nicht mehr zu nutzender Packages. Das solltest du dir vorher anschauen.

image

Ich habe mir nochmal tiefer eingelesen in die Warning Geschichte: Hier ein Beitrag dazu: facebook/create-react-app#11174 scheint dann eine Problematik mit npm zu sein. Soll ich dann react-scripts unter devDependencies anlegen, oder so lassen wie es aktuell ist?

href="/"
align="center"
sx={{ flexGrow: 1, color:'#FFFFFF' }}
<Link
Copy link
Contributor

Choose a reason for hiding this comment

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

Das sollte normalerweise über SCSS / CSS gesteuert werden. Inline-CSS wird nicht skalieren.

@saschadoemer saschadoemer merged commit e38443b into main Nov 8, 2024
@saschadoemer saschadoemer deleted the 1-1-anlegen-der-seite-impressum branch November 8, 2024 08:20
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.

1. Anlegen der Seite "Impressum"
2 participants