search

ArvidStrauss / dashboard-master

Vue.js Project that displays live Machine-Learning Prediction Data via Chart.js
2 stars 0 forks source link

Localhost Removal #8

Open CSchmidtDD opened 4 years ago

CSchmidtDD commented 4 years ago

Hey @ArvidStrauss & @Waescheklammer .

Durch die Hardkodierung von localhost kann man die Lösung nicht eigenständig laufen lassen. Im Kontext Docker ist das auch schwierig, deswegen bitte als Property in eine Konfiguration auslagern und den Pfad auslesen. https://github.com/ArvidStrauss/dashboard-master/search?p=1&q=localhost&unscoped_q=localhost

Außerdem habt ihr euch mehrmals beim Port vertan, mal 8000, mal 8080.

Waescheklammer commented 4 years ago

der localhost Teil ist ausgelagert in die urlConfig.json. Den Rest der Links ebenfalls variabel zu machen macht denke ich kaum Sinn, da einige vereinzelte höchstens 2x aufgerufen werden

CSchmidtDD commented 4 years ago

Referenz-Commit: 4ce726012648429ffff8b9bb41ef2a27cdeba3a6

Die Ausgestaltung finde ich bisher gut, kannst du das für die anderen Dateien mit der Direktferenz zu localhost auch noch übernehmen? Es finden sich hier immernoch non-config werte: https://github.com/ArvidStrauss/dashboard-master/search?q=localhost&unscoped_q=localhost

PS: Bitte bei den Commits die Issue Number mit angeben, dann wird die auch richtig hier rein zugeordnet und man muss nicht suchen. Danke :)