Update Podlove Player

[ltsavar]

Unser Webplayer müsste dringend mal geupdatet werden....

[fliiiix]

Der Podlove Web Player 4.0 https://docs.podlove.org/podlove-web-player/ könnte man integrieren.

[dev-matil]

Jo, also ich würd ja mal gern daran rumfrickeln nun ist mein Problem das ich null ruby erfahrung habe. Ich versuche mich gerade mit dem Buildsystem auseinander zu setzen. Frage -> könnt ihr evtl nen Dockerfile und die zwei commands für test/run bereitstellen. Damit ich schonmal den generator starten kann ohne nen udemy kurs für ruby zu belegen =)?

[dev-matil]

FROM ruby:2.5.1
ADD . /src
WORKDIR /src

RUN bundle install --jobs=3 --retry=3 --deployment --path=vendor/bundle
RUN bundle exec rake

Das wäre mein erster versuch (aus travis-ci extrahiert) leider mit folgendem issue im rake:

Step 5/5 : RUN bundle exec rake
 ---> Running in c8d131c84320
/usr/local/bin/ruby -w -I"lib:test" -I"/src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib" "/src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib/rake/rake_test_loader.rb" "tests/smoke_test.rb"
/usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.5/lib/bundler/definition.rb:559: warning: instance variable @new_platform not initialized
/usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.5/lib/bundler/definition.rb:535: warning: instance variable @new_platform not initialized
/usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.5/lib/bundler/definition.rb:535: warning: instance variable @new_platform not initialized
/src/vendor/bundle/ruby/2.5.0/gems/feedvalidator-0.2.1/lib/feed_validator/assertions.rb:71: warning: mismatched indentations at 'end' with 'begin' at 62
/src/vendor/bundle/ruby/2.5.0/gems/addressable-2.5.2/lib/addressable/idna/pure.rb:154: warning: assigned but unused variable - startercc
/src/vendor/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/builder.rb:36:in `[]': invalid byte sequence in US-ASCII (ArgumentError)
        from /src/vendor/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/builder.rb:36:in `parse_file'
        from /src/tests/smoke_test.rb:8:in `<top (required)>'
        from /src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib/rake/rake_test_loader.rb:17:in `require'
        from /src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib/rake/rake_test_loader.rb:17:in `block in <main>'
        from /src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib/rake/rake_test_loader.rb:5:in `select'
        from /src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib/rake/rake_test_loader.rb:5:in `<main>'
rake aborted!
Command failed with status (1): [ruby -w -I"lib:test" -I"/src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib" "/src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/lib/rake/rake_test_loader.rb" "tests/smoke_test.rb" ]
/src/vendor/bundle/ruby/2.5.0/gems/rake-12.3.1/exe/rake:27:in `<top (required)>'
/usr/local/bin/bundle:23:in `load'
/usr/local/bin/bundle:23:in `<main>'
Tasks: TOP => default => test
(See full trace by running task with --trace)
The command '/bin/sh -c bundle exec rake' returned a non-zero code: 1

Erwartungsgemäß ist mir natürlich teilweise unklar was hier passiert. Any suggestions? Ansonsten popel ich mal rum..

[fliiiix]

:tada: klingt gut ich mach mich nacher mal daran das in ein docker dingsi zu packen

[fliiiix]

ping @sukramlitt ein work in progress Dockerfile findest du hier: https://github.com/Binaergewitter/serious-bg/pull/276

docker build -t blog . && docker run -p 9292:9292 blog

Die relevanten files für den payer sind so:

• der live on air player: https://github.com/Binaergewitter/serious-bg/blob/master/serious/lib/site/views/_live.erb
• der reguläre player: https://github.com/Binaergewitter/serious-bg/blob/master/serious/lib/site/views/_podcast_controls.erb
• css & js kram in https://github.com/Binaergewitter/serious-bg/tree/master/serious/lib/site/public/podlove-web-player und https://github.com/Binaergewitter/serious-bg/tree/master/serious/lib/site/public/mediaelement der vermutlich weg kann

[dev-matil]

@fliiiix works for me. Ich schau dann mal was ich als k8s/DevOp so am frontend ausrichten kann ^^

[dev-matil]

@fliiiix #277 bitte schau dir mal #277 an und sag mir wie du es grundsätzlich findest. Für mich ist es aktuell noch ein Problem das euer Content nicht zu dem fließtext passt den der player erwartet. Die Frage ist wie gehen wir damit um, wir könnten für alte folgen die klassische Ansicht rendern also als plain html + neuer player. Oder diese Ansicht bleibt der detail view vorbehalten.

Man könnte ja den kontent der letzten n folgen anpassen so das es im player mehr Sinn macht. Alternative können wir auch einfach das alte layout für den content behalten.

[dev-matil]

und wann wandert das in prod =)?

[dev-matil]

okay instant ^^. Hoffe es gefällt soweit

eine Sache ist defentive noch offen: die file sizes im download ist immer 100mb weil ihr keine angebt. Ansonsten bitte bugs melden ich schau dann drauf

[fliiiix]

@rb2k oder @pfleidi habt ihr vorschläge wie man die filesize am besten bekommt?

[dev-matil]

Evtl. Script ich das nachher schnell runter..

[ltsavar]

hatte nicht @rb2k mal was gemacht. siehe #177

[dev-matil]

278 ich hab mal den liverplayer auch implementiert.. ist mir vorher entfallen und tauchte auch nirgends auf ¯_(ツ)_/¯

[dev-matil]

@ltsavar guter hinweis hab ich direkt integriert die file size problematik sollte mit #278 auch erschlagen sein

[dev-matil]

279 kleiner fix für die responsive ui

[fliiiix]

@sukramlitt das ist noch ein bug oder?

[ltsavar]

Ja die Info haben wir nicht bzw. hat noch nie jemand gebaut Siehe auch #178 das müsste man berechnen.... Alternativ gibt es zu den letzten 100 Sendungen oder mehr immer eine .json Datei: http://download.binaergewitter.de/2016-09-01.Binaergewitter.Talk.156.json Da steht auch die Länge drin....

[fliiiix]

Joa die letzen 100 reichen glaub ich dann kann man was bauen was jeweils die Länge mit js aus dem file holt

[dev-matil]

@fliiiix Danke für den schönen Twitter post. Ich schaue Mal ob ich die Woche noch Zeit finde für die Spieldauer. Aber wenn das erstmal das einzige ist dann bin ich schonmal ganz zufrieden.

[fliiiix]

281 ist noch offen aber der neue player ist migriert und läuft :tada: