Skip to content
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

Toggle repeat on/off #21

Closed
jiri-janousek opened this issue Nov 10, 2014 · 15 comments
Closed

Toggle repeat on/off #21

jiri-janousek opened this issue Nov 10, 2014 · 15 comments
Assignees
Labels
type: enhancement A new feature or an improvement of already present feature.

Comments

@jiri-janousek
Copy link
Member

Integrate player action toggle repeat on/off

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@ghost ghost assigned jiri-janousek Feb 6, 2017
@ghost ghost removed the postponed label Feb 6, 2017
@jiri-janousek jiri-janousek reopened this Feb 6, 2017
@jiri-janousek jiri-janousek added type: enhancement A new feature or an improvement of already present feature. status: backlog labels Jul 2, 2018
jiri-janousek added a commit that referenced this issue Sep 10, 2018
Issue: #21

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit to tiliado/nuvolasdk that referenced this issue Sep 13, 2018
jiri-janousek added a commit that referenced this issue Sep 13, 2018
Property change notification was not sent.

Issue: #21

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit that referenced this issue Sep 14, 2018
Issue: #21

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit to tiliado/nuvola-app-deezer that referenced this issue Sep 15, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Jiří Janoušek <[email protected]>
@jiri-janousek
Copy link
Member Author

Hello @a-m-s, @bors-ltd, and @d-r-q. Could you integrate the repeat status if it is relevant to your scripts? You need up-to-date Nuvola ADK >= 4.12.86. Also, don't forget to raise api_major/api_minor in metadata.in.json to 4/13. Thanks!

jiri-janousek added a commit to tiliado/nuvola-app-spotify that referenced this issue Sep 16, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit to tiliado/nuvola-app-soundcloud that referenced this issue Sep 16, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit to tiliado/nuvola-app-google-play-music that referenced this issue Sep 16, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Jiří Janoušek <[email protected]>
@a-m-s
Copy link

a-m-s commented Sep 17, 2018

Amazon Music is done. The others don't have a repeat feature.

jiri-janousek added a commit that referenced this issue Sep 17, 2018
Issue: #21

Signed-off-by: Jiří Janoušek <[email protected]>
jiri-janousek added a commit to tiliado/nuvolasdk that referenced this issue Sep 17, 2018
@d-r-q
Copy link

d-r-q commented Sep 18, 2018

Yandex.Music don't support repeat

@jiri-janousek
Copy link
Member Author

Amazon Music is done. The others don't have a repeat feature.

Thanks :-) I've added a few inline comments to address: tiliado/nuvola-app-amazon-cloud-player@74fa997

@jiri-janousek
Copy link
Member Author

Yandex.Music don't support repeat

Hello @d-r-q. There seems to be a repeat button in the bottom right corner (next to the volume button) on your screenshot. Was it removed? Or what's its functionality?

@d-r-q
Copy link

d-r-q commented Sep 19, 2018

Oh, I'm blind... Integrate it later this week.

jiri-janousek added a commit to tiliado/nuvola-app-plex that referenced this issue Sep 21, 2018
jiri-janousek added a commit to tiliado/nuvola-app-owncloud-music that referenced this issue Sep 21, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Jiří Janoušek <[email protected]>
@jiri-janousek
Copy link
Member Author

Currently completed repeat integrations will be announced on Sunday, September 23rd. The rest with Nuvola 4.13.0 (#472) or later when implemented.

d-r-q pushed a commit to d-r-q/nuvola-app-yandex-music that referenced this issue Sep 23, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Aleksey Zhidkov <[email protected]>
d-r-q pushed a commit to d-r-q/nuvola-app-yandex-music that referenced this issue Sep 23, 2018
Issue: tiliado/nuvolaplayer#21

Signed-off-by: Aleksey Zhidkov <[email protected]>
@d-r-q
Copy link

d-r-q commented Sep 23, 2018

Hi @fenryxo, I've added support for repeat (and shuffle), but I'm little confused with current release process. Should I create an issue and/or pull request for this changes? And should I update changelog file?

bors-ltd pushed a commit to tiliado/nuvola-app-qobuz that referenced this issue Sep 23, 2018
@bors-ltd
Copy link

Added both repeat a track and the whole playlist to Qobuz.

Updated the changelog, ready for release.

I forgot to mention this on the shuffle issue but for some reason, the functions setCanShuffle and setCanRepeat were not available, though I used what I think is the latest SDK 4.12.92. So I used the "low-level" ones from the first announce. It won't make a difference for users.

@jiri-janousek
Copy link
Member Author

I've added support for repeat (and shuffle), but I'm little confused with current release process. Should I create an issue and/or pull request for this changes? And should I update changelog file?

Hello @d-r-q. The current release process needs to be documented, because the one in docs predates the switch to Flatpak builds. It works roughly this way:

  • Always update the changelog, but don't touch version numbers in metadata.in.json.
  • If you wish to have a code review, create a pull request to the master branch.
  • If you don't need a code review, commit directly to the master branch (@a-m-s does that).
  • Whatever is in the master branch is included in the next rebuild of Flatpaks.
  • Once for a while, I make a new X.Y.0 release of Nuvola and all components for third-party packagers. That's when the version_minor field gets updated.

@jiri-janousek
Copy link
Member Author

Added both repeat a track and the whole playlist to Qobuz.

Updated the changelog, ready for release.

Great, I rebuilt Qobuz flatpak today in the morning so your changes are live :-)

I forgot to mention this on the shuffle issue but for some reason, the functions setCanShuffle and setCanRepeat were not available, though I used what I think is the latest SDK 4.12.92. So I used the "low-level" ones from the first announce. It won't make a difference for users.

That's strange. Are you sure that nuvolaruntime --version said 4.12.92? If you are in a Bash session of Nuvola ADK and you update the ADK flatpak, the old flatpak is still in use until all its instances exit. (This way, it won't happen that a running flatpaked app crashes because its data files have suddenly changed because of a pending update.)

@jiri-janousek
Copy link
Member Author

@bors-ltd
Copy link

Here is the output:

$ nuvola-adk
bash-4.3$ nuvolaruntime --version
Qobuz script
Version: 1.1.3
Revision: 3-gca69c9a
Maintainer: Bors, Ltd

--- Powered by ---

Nuvola Apps - Genuine flatpak build
Version 4.12.92
Revision 92-gbe08264
Diorite 4.12.24
WebKitGTK 2.20.2
Chromium 68.0.3440.75
libsoup 2.62.1

I'm sure no ADK session was open when I updated. I'll try again when updating to 4.13.0.

@jiri-janousek jiri-janousek added release notes The work on the ticket is ready, but needs to be released and announced. and removed status: in progress labels Oct 14, 2018
@jiri-janousek
Copy link
Member Author

Hello @bors-ltd, could you check whether your issue persists with Nuvola ADK 4.13.0? If yes, please create a new ticket for that. Thanks.

@jiri-janousek
Copy link
Member Author

Released in Nuvola 4.13.0.

@ghost ghost removed the release notes The work on the ticket is ready, but needs to be released and announced. label Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A new feature or an improvement of already present feature.
Projects
None yet
Development

No branches or pull requests

4 participants