YunoHost-Apps / peertube_ynh

Federated (ActivityPub) video streaming platform using P2P for YunoHost
https://joinpeertube.org/fr/
GNU Affero General Public License v3.0
88 stars 28 forks source link

Update upgrade #419

Closed ericgaspar closed 8 months ago

ericgaspar commented 9 months ago

https://github.com/YunoHost-Apps/peertube_ynh/pull/418#issuecomment-1878469262

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

ericgaspar commented 9 months ago

!testme

yunohost-bot commented 9 months ago

Fingers crossed! Test Badge

lapineige commented 9 months ago

On upgrade from CLI I have this:

  • peertube can now be accessed at URL
  • The admin username is: root
  • The admin password is: PWD

It works :)

But... that mean it resets my root account password ! Why would we do that ??

lapineige commented 8 months ago

Why did you close ? :thinking:

ericgaspar commented 8 months ago

I didn't, I moved to a specific branch change url so it can be tested without blocking testing

ericgaspar commented 8 months ago

Well, now we have two change URL branch...

lapineige commented 8 months ago

I don't understand… how it that PR related to change URL ?

lapineige commented 8 months ago

I didn't, I moved to a specific branch change url so it can be tested without blocking testing

Also I don't understand why there is now a specific branch (that I wasn't aware of) while it was already in testing ? https://github.com/YunoHost-Apps/peertube_ynh/pull/422 And it wasn't advertise anywhere :sweat_smile:, while we originally had #420 with its custom branch dedicated to that purpose ? :thinking: