Details des Tickets

Beschreibung,Kommentare und Anhänge

merge request id 56
TitelFeature/cookie warning
Beschreibung
Statusmerged
Created at 2019-01-09T08:20:00.660Z
Updated at 2019-01-09T16:15:19.930Z
Source branchfeature/cookie-warning
Target branchdevelop
AuthorSascha Immig
AssigneeSebastian Schlein

Discussion notes

Macht es Sinn hier den relativen statt des absoluten Links zu nutzen? Ich bin beim testen darauf reingefallen, dass der Hinweis nicht auf den anderen Seiten kam, bis ich gemerkt habe, dass ich auf dem Prod System unterwegs war. Vermutlich wird sich die URL nicht mehr ändern, aber wenn doch, wäre hier keine Anpassung notwendig. Alternativ könnte man hier auf {{ config('app.url') }}/datenschutz zurückgreifen, sodass im HTML der komplette Link ankommt, aber auf Testsystemen der Pfad dennoch stimmt. Dann muss nur jeder darauf achten, dass auf allen Systemen die APP_URL in der .env richtig gesetzt ist (was sie eh sein sollte). Sebastian Schlein 2019-01-09T09:21:04.221Z
absolut richtig, korrigiert! :) Sascha Immig 2019-01-09T12:58:58.051Z
resolved all discussions Sascha Immig 2019-01-09T13:04:57.423Z
changed this line in version 2 of the diff Sascha Immig 2019-01-09T13:05:37.715Z
added 1 commit
  • c3d9ef5f - fixed URL by using config
Compare with previous version
Sascha Immig 2019-01-09T13:05:37.907Z
merged Sascha Immig 2019-01-09T16:15:19.981Z

Merge Commits

Title Id Author Created at Message
added cookie warning via jquery 21b39a69b1ec2c3b00d96a092bf3d99b79c45129 TUV\immigs 2019-01-08T15:52:17.000Z added cookie warning via jquery
optimized sticky-box-script 6d2cae8431a27d986490505f54983e0bdade6316 TUV\immigs 2019-01-08T16:09:00.000Z optimized sticky-box-script
fixed URL by using config c3d9ef5f043da32bd43b555b6cf8cd6c526308a4 TUV\immigs 2019-01-09T13:04:43.000Z fixed URL by using config