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

Implemented high-volume events, dynamic reidentify, new event protocol #92

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dedmen
Copy link
Contributor

@dedmen dedmen commented Jul 28, 2022

Implemented events for protocol v5 ("update-type" changes to "eventType" and event data is in "eventData" member)
Implemented InputVolumeMeters event as a example of a high-volume event that requires dynamic subscription (via Reidentify) for the event

By the way your discord link in readme isn't working.

Copy link
Owner

@BarRaider BarRaider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the "_" from variables to align with the existing variable naming conventions in the project.

@dedmen
Copy link
Contributor Author

dedmen commented Sep 3, 2022

Sorry don't have time now to fix the conflicts, maybe tomorrow or next week

@dedmen dedmen force-pushed the patch-NewEvents branch 2 times, most recently from 78e9155 to d8f7492 Compare October 1, 2022 11:29
@dedmen dedmen changed the base branch from v5-dev to master October 1, 2022 11:30
@dedmen
Copy link
Contributor Author

dedmen commented Oct 1, 2022

Okey here we go, that should be it.
Tested working.

@TabNoc
Copy link

TabNoc commented Dec 26, 2022

I'm interested in this feature, any updates?

@BarRaider
Copy link
Owner

BarRaider commented Dec 28, 2022

I was planning on adding it a while back but there are some uncomplete ("Todo") sections in the identify section as well as breaking backwards compatibility for the events that prevented me from pushing it.
I'll look into taking the Volume parts of the PR and adding that specifically

@dblanco215
Copy link

Hi! I'm trying to implement this feature on my app in C#. I'm add the event subscription but I never receive any data of the volume of OBS. Are there any additional step to can receive this data?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants