-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Improve documentation on web rtc #6005
Conversation
✅ Deploy Preview for frigate-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
docs/docs/configuration/live.md
Outdated
services: | ||
frigate: | ||
ports: | ||
- "8555:8555" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably match the config in the main docker getting started docs and suggest mapping 8555 tcp and 8555 udp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great suggestions! Thank you for reviewing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, what I meant was you should take the way it is shown there (where it maps tcp and udp) and apply that to your changes in live.md
Co-authored-by: Nicolas Mowen <nickmowen213@gmail.com>
This improves some documentation on getting web rtc working, especially when running frigate on different host machine. Please let me know if you have any further improvements or suggestions.