-
-
Notifications
You must be signed in to change notification settings - Fork 949
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
Feature: OpenWRT service widget #2782
Conversation
I'm not 100% on the statistics shown here, though the OpenWRT rpc schema is a little tricky to work with so thought of this as decent enough start. Please let me know if you have any suggestions or any requirements I have missed. Thanks |
Please provide some sample data, thanks |
Not sure if this is what you want exactly, let me know if not. Example
Example response data for
|
Yea thanks, I dont have an openwrt device to test. Is the |
Yeah, basically when first logging in you need to provide a ubus_rpc_session as the first element in params array (just as when you're logged in)
|
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.
Ah, sorry my bad I did not mean to actually push that changes (I had them just to test the lint). |
All good~ I've pushed up with your suggestions. Only one I differed on was using Thanks! |
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 thanks
Thanks shamoon |
* Feat: OpenWRT widget implementation * Update proxy.js * fixes from review --------- Co-authored-by: shamoon <4887959+shamoon@users.noreply.github.com>
* Feat: OpenWRT widget implementation * Update proxy.js * fixes from review --------- Co-authored-by: shamoon <4887959+shamoon@users.noreply.github.com>
* Feat: OpenWRT widget implementation * Update proxy.js * fixes from review --------- Co-authored-by: shamoon <4887959+shamoon@users.noreply.github.com>
* Feat: OpenWRT widget implementation * Update proxy.js * fixes from review --------- Co-authored-by: shamoon <4887959+shamoon@users.noreply.github.com>
* Feat: OpenWRT widget implementation * Update proxy.js * fixes from review --------- Co-authored-by: shamoon <4887959+shamoon@users.noreply.github.com>
Better to start a new discussion than comment on a closed PR. Also please post the container logs and api output (there) for that |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns. See our contributing guidelines for more details. |
Proposed change
Adds a widget for OpenWRT integration
Allows for either system info or information on a particular interface (i.e eth0 for WAN) by adding that interface name as
interfaceName
in services.yamlCloses #1509
Type of change
Adds a new widget for OpenWRT
Checklist: