New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Podlove Player #271

Closed
ltsavar opened this Issue Sep 9, 2018 · 20 comments

Comments

Projects
None yet
3 participants
@ltsavar
Member

ltsavar commented Sep 9, 2018

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

@ltsavar ltsavar added the enhancement label Sep 9, 2018

@fliiiix fliiiix added the help wanted label Sep 9, 2018

@fliiiix

This comment has been minimized.

Contributor

fliiiix commented Sep 9, 2018

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

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Sep 29, 2018

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 =)?

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Sep 29, 2018

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

This comment has been minimized.

Contributor

fliiiix commented Sep 30, 2018

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

@fliiiix

This comment has been minimized.

Contributor

fliiiix commented Sep 30, 2018

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

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

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

@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.

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

und wann wandert das in prod =)?

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

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

This comment has been minimized.

Contributor

fliiiix commented Oct 1, 2018

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

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

Evtl. Script ich das nachher schnell runter..

@ltsavar

This comment has been minimized.

Member

ltsavar commented Oct 1, 2018

hatte nicht @rb2k mal was gemacht. siehe #177

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

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

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 1, 2018

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

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 2, 2018

#279 kleiner fix für die responsive ui

@fliiiix

This comment has been minimized.

Contributor

fliiiix commented Oct 2, 2018

@sukramlitt das ist noch ein bug oder?

2018-10-02--1538485651_3200x1800

@ltsavar

This comment has been minimized.

Member

ltsavar commented Oct 2, 2018

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

This comment has been minimized.

Contributor

fliiiix commented Oct 2, 2018

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

@sukramlitt

This comment has been minimized.

Contributor

sukramlitt commented Oct 2, 2018

@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

This comment has been minimized.

Contributor

fliiiix commented Oct 13, 2018

#281 ist noch offen aber der neue player ist migriert und läuft 🎉

@fliiiix fliiiix closed this Oct 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment