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

[FEATURE]: YTMD v2 Support #85

Closed
2 tasks done
ElectricalBoy opened this issue Sep 26, 2023 · 13 comments · Fixed by #98
Closed
2 tasks done

[FEATURE]: YTMD v2 Support #85

ElectricalBoy opened this issue Sep 26, 2023 · 13 comments · Fixed by #98
Assignees
Labels
Priority: Critical If a issue is in critical state and needs to be resolved ASAP Special: Good first issue Special "achievement" for good first issues Status: Accepted Feature got accepted Status: In Progress Developer is working on the issue. Type: Enhancement Feature request

Comments

@ElectricalBoy
Copy link

Consent

  • I have searched the existing issues and verified that there is no open issue for the same subject.
  • I understand that the YTMDesktop Elgato Plugin has NO affiliation with Google, YouTube or the YTMDesktop application.

Feature

Context: ytmdesktop/ytmdesktop#1139

YTMDesktop (YTMD) has announced v2 (see above) and v2 is in active development. The changes of v2 include a backwards-incompatible companion server. This implies that the current version of the plugin would be incompatible with YTMD v2. This is a feature request for YTMD v2 support.

Solution

YTMD v2 compatibility of this plugin

Alternatives

N/A

Anything else?

API documentation of the new companion server

@ElectricalBoy ElectricalBoy added the Type: Enhancement Feature request label Sep 26, 2023
@XeroxDev
Copy link
Owner

Hey and thank you for opening this issue!

I'm already aware of this and am active following the development (that's why there wasn't an update in quite a time). After they implemented an (suffice) working server, I'll updating this plugin as well and will only support v2+ in the future.

I'll keep this issue open until the switch happened. Thanks again!

/label +good +hold +critical

@slash-me slash-me bot added Priority: Critical If a issue is in critical state and needs to be resolved ASAP Special: Good first issue Special "achievement" for good first issues Status: On Hold Issue is on hold cause of internal or external reasons. labels Sep 26, 2023
@XeroxDev XeroxDev pinned this issue Sep 26, 2023
@ZoinktheBoink
Copy link

ZoinktheBoink commented Jan 24, 2024

Hey man, love this plugin. Any word from the devs regarding the perpetuity of the API?

https://github.com/ytmdesktop/ytmdesktop/wiki/v2-‐-Companion-Server-API-v1

@XeroxDev
Copy link
Owner

Hey man, love this plugin. Any word form the devs regarding the perpetuity of the API?

https://github.com/ytmdesktop/ytmdesktop/wiki/v2-‐-Companion-Server-API-v1

Thanks for the feedback! I'll contact the developer of YTMDesktop soon and ask if the endpoints / new api is still subject to change.

Depending on the response I'll keep waiting or start developing the new version.

Tbh: I'm not the biggest fan of the authorization change, especially because it's a private / (sub-) local service and not a public one (IMO it's a bit redundant and over-engineered / overkill for such an application :D This just adds more complexity for the developers and users)

@jared-craig
Copy link

Any word on when this will be updated to work with v2.0? Thanks!

@XeroxDev
Copy link
Owner

Any word on when this will be updated to work with v2.0? Thanks!

Sure. I already posted an announcement on Sunday on my discord server but I've forgotten to do it also here. I'm sorry! This is what I wrote:

image

But now I can say a bit more: I've already developed a new library for the new server. I just need to test it a bit further and then I can implement it in the plugin itself, build the updated artifact and send it to elgato. After that, it should be on the store very soon!

@XeroxDev XeroxDev added Status: Accepted Feature got accepted Status: In Progress Developer is working on the issue. and removed Status: On Hold Issue is on hold cause of internal or external reasons. labels Jan 30, 2024
@XeroxDev XeroxDev self-assigned this Jan 30, 2024
@ZoinktheBoink
Copy link

But now I can say a bit more: I've already developed a new library for the new server. I just need to test it a bit further and then I can implement it in the plugin itself, build the updated artifact and send it to elgato. After that, it should be on the store very soon!

This is great news, thanks @XeroxDev.

@XeroxDev
Copy link
Owner

XeroxDev commented Feb 5, 2024

Hey everyone!

I believe v2 support is done! I won't publish it today though (it's 10:45 PM for me)

But I've created an artifact in the Pull Request. See: #98 (comment)

Like mentioned there, you can, if you want, also build it yourself!

Feel free to test it and send feedback here (as long as the issue is still open :D)

For an updated "How to use" / README, refer to https://github.com/XeroxDev/YTMD-StreamDeck/blob/feat/v2/README.md as long as it's not merged yet.

Hope y'all have a good day/noon/evening/night or what ever time you've lol!

@digihash
Copy link

digihash commented Feb 6, 2024

Hey everyone!

I believe v2 support is done! I won't publish it today though (it's 10:45 PM for me)

But I've created an artifact in the Pull Request. See: #98 (comment)

Like mentioned there, you can, if you want, also build it yourself!

Feel free to test it and send feedback here (as long as the issue is still open :D)

For an updated "How to use" / README, refer to https://github.com/XeroxDev/YTMD-StreamDeck/blob/feat/v2/README.md as long as it's not merged yet.

Hope y'all have a good day/noon/evening/night or what ever time you've lol!

Thanks a lot @XeroxDev! This plugin together with YTMD makes it so easy to control music. I just tested the build version in #98 (comment) and it works perfectly :)

@jared-craig
Copy link

Hmm. I was using the new artifact just fine but it stopped working for me after I restarted YTMD. It's still listed under authorized companions, and I've even re-authorized. It was working great, but now it seems broken for me.

@XeroxDev
Copy link
Owner

XeroxDev commented Feb 6, 2024

Hmm. I was using the new artifact just fine but it stopped working for me after I restarted YTMD. It's still listed under authorized companions, and I've even re-authorized. It was working great, but now it seems broken for me.

Mh... That's weird.. I tried to reproduce it but everything worked for me.

I tried the following things:

First:

  1. Have StreamDeck and YTMDesktop started (and authorized/connected)
  2. Close YTMDesktop
    • Play/Pause shows warn icon (triangle)
  3. Wait for a few seconds
  4. Open YTMDesktop
    • Play/Pause shows success icon (green circle with check)

Second:

  1. Have both closed
  2. First start Stream Deck
    • Play/Pause shows warn icon (triangle)
  3. Start YTMDesktop
    • Play/Pause shows success icon (green circle with check)

And of course vice versa.

And everything worked.

Could you tell me what you did? Or what you did different?

@jared-craig
Copy link

Hmm. I was using the new artifact just fine but it stopped working for me after I restarted YTMD. It's still listed under authorized companions, and I've even re-authorized. It was working great, but now it seems broken for me.

Mh... That's weird.. I tried to reproduce it but everything worked for me.

I tried the following things:

First:

  1. Have StreamDeck and YTMDesktop started (and authorized/connected)

  2. Close YTMDesktop

    • Play/Pause shows warn icon (triangle)
  3. Wait for a few seconds

  4. Open YTMDesktop

    • Play/Pause shows success icon (green circle with check)

Second:

  1. Have both closed

  2. First start Stream Deck

    • Play/Pause shows warn icon (triangle)
  3. Start YTMDesktop

    • Play/Pause shows success icon (green circle with check)

And of course vice versa.

And everything worked.

Could you tell me what you did? Or what you did different?

Restarting the Stream Deck application seemed to fix the issue without any other steps taken. Thanks.

@ZoinktheBoink
Copy link

But I've created an artifact in the Pull Request. See: #98 (comment)

Works like a charm, well done!

@XeroxDev
Copy link
Owner

Hey! I've finally created the release! I will immediately contact elgato, so they can also release it the plugin store.

I'll also unpin this issue now. Have fun listening everybody!

@XeroxDev XeroxDev unpinned this issue Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Critical If a issue is in critical state and needs to be resolved ASAP Special: Good first issue Special "achievement" for good first issues Status: Accepted Feature got accepted Status: In Progress Developer is working on the issue. Type: Enhancement Feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants