8000 Add custom live pages/controllers/rest api for nerves only by gBillal · Pull Request #686 · evercam/ex_nvr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add custom live pages/controllers/rest api for nerves only #686

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

Merged
merged 2 commits into from
May 13, 2025

Conversation

gBillal
Copy link
Member
@gBillal gBillal commented May 11, 2025

This is an ugly solution for extending the ExNVR UI with live pages/controllers/rest API for nerves. The idea is to not make ExNVR UI extendable or introduce any fancy plugin systems for now. Since the nerves image is tailored for Evercam use cases, we want to hide those from the open source/generic NVR.

For this, we'll only add a compile environment variable nerves_routes, if set all the additional pages/controllers/rest APIs will be included. The side bar menus will also be controlled by a simple: Application.get_env(:ex_nvr, :nerves_routes). And I suspect we can do the same thing with other assets (javascript, css, ...etc).

If the need rise, we may create a proper plugin system.

@halimb can you give it a try !!

@gBillal gBillal requested a review from halimb May 11, 2025 10:40
Copy link

📦 To trigger a firmware build, comment /build below.
To build a specific target, comment /build <target> with on of the following targets
Valid target names:

  • ex_nvr_rpi4
  • ex_nvr_rpi5
  • giraffe

You can also specify a custom version: /build <target> version=<version>
Examples:
/build version=1.0.0-test
/build ex_nvr_rpi5 version=1.0.0-test.123

If no version is specified, one will be generated automatically.

@halimb
Copy link
Member
halimb commented May 11, 2025

Good, this achieves the desired outcome, with minimal / straightforward changes.
I'll give it a try in the coming days and push to this branch 👍

@gBillal gBillal merged commit 2fce5b7 into master May 13, 2025
@gBillal gBillal deleted the nerves-custom-routes branch May 13, 2025 16:16
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
0