search

marioneubacher1 / WebApp

0 stars 0 forks source link

Seite "news" #2

Open marioneubacher1 opened 9 months ago

marioneubacher1 commented 9 months ago

Da ich bereits eine API Einbindung mit dem BTC Kurs implementiert habe, ist eine zweite API Einbindung der Newsseite nicht nötig, richtig? Ich lasse diese dann weg, weil eine News Seite soll ja nicht statisch sein und die Mindestanzahl an Seiten ist erfüllt.

oliver-jung commented 9 months ago

Der Umfang der API-Anbindung ist jetzt nicht enorm. Außerdem gibt es beim Plotting scheinbar nach wie vor Probleme (siehe Errorlog anbei). Ich würde dazu raten, dass der Chart funktioniert und es darüber hinaus noch eine zusätzliche, etwas umfangreichere Funktion mit client- oder serverseitiger Programmierung gibt. (z.B. die News-Seite, ein echter Vergleich der Assets anstelle des statischen Bildes, ein funktionierendes Mailingscript, eine echte Nutzerauthentifizierung, o.ä.). Aktuell hängt noch ein bisschen zu viel unimplementiert in der Luft. error_chart

marioneubacher1 commented 9 months ago

Ich habe mein gesamtes Projekt nochmal committed. Bitte um Check folgender Änderungen:

oliver-jung commented 9 months ago

Ich bin nur kurz mit dem Handy eingewählt und kann daher nicht sinnvoll testen.

Generell ist es an dir sicherzustellen, dass die Funktionalitäten deiner Seite auch außerhalb deiner Testumgebung funktionieren. Ich rate dringend dazu das vor der finalen Abgabe zu testen, da ich auf derartige Versäumnisse keine Rücksicht nehmen kann.

Die Mindestanforderungen / den Erwartungshorizont habe ich nun schon einige Male abgesteckt. Um nicht gerade so an der Grenze zum Bestehen zu kratzen, ist es sicher förderlich diesen etwas zu übertreffen und die einzelnen Bestandteile sauber wie in VO und LB besprochen umzusetzen. Ich bin sicher kein überstrenger Benoter, aber verteile auch keine Freifahrtscheine. Entsprechend ECTS-Umfang sollte das Projekt einen Arbeitsaufwand von ca. 50-60 Stunden aufweisen wobei ihr ohnehin bereits einen guten Teil der LB-Einheiten von mir dafür erhalten habt, um die Last auf die "freie Zeit" zu minimieren.

Es gab das gesamte Semester Zeit für Detailfragen und Feedback außerhalb des Kurses. Jegliche Leistungen meinerseits außerhalb der LV-Zeit sind wohlbemerkt freiwillig und unbezahlt - sofern dies auf studentischer Seite nicht bekannt ist. Nun ist auf jeden Fall Weihnachtsurlaub und ich werde eine geruhsame Zeit mit Frau und Kindern im Rahmen der Familie verbringen und in diesem Zusammenhang sämtliche Arbeits- und FH-Mails abdrehen. Dies gilt grundsätzlich bis zur Abgabe wobei ich ab Neujahr auf wirklich dringende Fälle antworten werde sofern etwas Luft bleibt - dringend bedeutend nicht selbst lösbar und/oder in den LV-Unterlagen zu finden.

Schöne Feiertage!

marioneubacher1 commented 9 months ago

Meiner Nutzerauthentifizierung will nicht klappen, dieses Problem beschäftigt mich bereits mehrere Stunden. Der Code in meinen letzten Commit sollte jedoch passen. Sollte ich nochmal mit dem auskommentierten Code committen, damit die Funktionalität der Seite gewährleistet ist, oder soll ichs drin lassen, für eine bessere Benotung? Schönen Feiertag

oliver-jung commented 9 months ago

Ich hoffe sehr die 10 Zeilen Code des letzten Commits haben nicht mehrere Stunden Arbeit verursacht...

Von Authentifizierung ist das Ganze aus verschiedenen Gründen weit entfernt. Grundsätzlich kann deine aktuelle Umsetzung nicht funktionieren, da es keine Datenpersistenz gibt. Die Nutzerinformationen werden in deiner aktuellen Implementierung in eine lokale Variable geschrieben. Diese verliert sofort beim Redirect auf die Login-Page ihre Gültigkeit. Sauber gelöst wäre dies mit einer eigenen kleinen Datenbank (z.B. MariaDB, die mit XAMPP geliefert wird). Zu Demozwecken bekommst du deinen aktuellen Ansatz vermutlich am schnellsten lauffähig, wenn du den Web Storage verwendest - auch wenn dieser nur für einen Nutzer (und eine Session) gilt und somit natürlich in der Praxis zur Persistierung von Nutzerinformationen absolut ungeeignet wäre: https://www.w3schools.com/html/html5_webstorage.asp

Alternativ lassen sich relativ schnell und einfach offene OAuth(2) Provider anbinden - GitHub selbst bietet hierzu ebenfalls eine Implementierung: https://docs.github.com/de/apps/oauth-apps/building-oauth-apps/creating-an-oauth-app Die Liste an Anbietern ist entsprechend lang: https://en.wikipedia.org/wiki/List_of_OAuth_providers

Allgemein bitte auf keinen Fall Code abgeben, der die Seite zum Absturz bringt! Im Zweifelsfall auskommentiert abgeben und im Kommentar erklären was die Intention war und wo man angestanden ist.