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

223 metadata injector #252

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from
Draft

223 metadata injector #252

wants to merge 4 commits into from

Conversation

jr1221
Copy link
Contributor

@jr1221 jr1221 commented Nov 29, 2024

Changes

Adds a metadata injector. It basically splits off socket functionality from mqtt_processor into its own file, and hides the metadata behind a flag.

Notes

  • Currently only viewer count implmented metadata type

To Do

Any remaining things that need to get done

Checklist

  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • No package-lock.json changes (unless dependencies have changed)
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #223

@jr1221 jr1221 force-pushed the 223-metadata-injector branch from dc6bd5f to 128b451 Compare December 20, 2024 02:58
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.

[Scylla] - Add Scylla Metadata Injection
1 participant