search

YunoHost-Apps / mopidy_ynh

Mopidy package for YunoHost
http://www.mopidy.com
Apache License 2.0
4 stars 7 forks source link

Can't install app without providing Spotify username. #2

Closed elvirolo closed 7 years ago

elvirolo commented 7 years ago

Mopidy cannot be installed without providing a Spotify username. This should not be compulsory.

frju365 commented 7 years ago

Hello, Thanks for reporting it, That will surely be solved in my PR : https://github.com/YunoHost-Apps/mopidy_ynh/pull/3 frju365

elvirolo commented 7 years ago

Hi! Your PR didn't work. I made another one which does #4 . Thanks for showing me where to look at :)

frju365 commented 7 years ago

In fact, my PR works fine. :~ (I tested it)

Do yunohost app install https://github.com/YunoHost-Apps/mopidy_ynh/tree/Little-correction in a terminal and you will see that it works.

frju365 commented 7 years ago

Of course my PR was not committed yet. It's a PR. I'm not the maintainer of the package, so I need @magikcypress to approve my PR before merging it (even if it works).

elvirolo commented 7 years ago

You're right, my mistake. Sorry about that.

frju365 commented 7 years ago

No problem :)