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

Enable editing of ONVIF URL IP address via the Edit Broadcast dialog #216

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lastpeony
Copy link

Before, any changes made to the IP address of an ONVIF camera during editing were not saved, and the old IP address was consistently used. With this update, if it is possible to establish a connection to the camera using the new IP address, it will be saved and utilized.

@mekya
Copy link
Contributor

mekya commented Jun 20, 2023

Hi @lastpeony ,

Thank you for the fix. I just need some clarity about this PR.

I think something in the UI is confusing and it leads to further confusion.

Generally, the expectation is to connect to the IP Camera through it's ONVIF address and get the stream URL from the IP Camera automatically.

With this development, ONVIF address and stream URL ar changing independently. I just need some clarity to proceed with this PR.

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.

2 participants