Skip to content
Snippets Groups Projects

closes #187

Closed Jan Philipp Hafer requested to merge jph/videoagwebsite:master into master

setting all 10 seconds simple cookie to save currentTime of video in seconds and resets it if time lower than actual time; upon reponening players starts again on that time

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Reassigned to @andreasv

  • danke fürs Implementieren lies dir nochmal am besten #176 (closed) durch

    hab außerdem noch ein paar Ideen/Kriterien hierfür: zum einen ist es nicht sinnvoll für ggf. hunderte Videos jedesmal alle Cookies mit zum Server zu schicken -> indexed db?

    zum anderen reicht es den Cookie zu setzen, wenn die Seite verlassen wird -> onunload event oder ggf. jede Minute, falls da Browser doch Probleme haben sollten

    der Cookie sollte gelöscht werden, sobald der Nutzer mehr als 90/95% des Videos angesehen hat und das Cookie sollte nicht länger als/bis zu einem Semester leben (falls wir bei Cookies bleiben)

    Die Funktion sollte optional sein und dem Nutzer vorgestellt werden => man könnte eine Seite anbieten auf der man die Cookies auch löschen kann.

    für jetzt reicht die Funktion erstmal aber, das kann man mit einem folgenden request nachreichen

  • werde das vorm mergen noch auf webstore umstellen. eine eigene datenbank halte ich für nicht sinnvoll. es spricht nichts dagegen das die ganze zeit zu setzen, cachen kann das der browser. das mit dem löschen ist aber allerdings wahr

  • ja, an webstorage hatte ich nicht gedacht, ist tatsächlich sinnvoller

  • hab eine eigene Lösung mittels localstorage gebaut, siehe 9c350669

  • closed

Please register or sign in to reply
Loading