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

ODE-128 Added snmp functionality and web UI component #31

Conversation

mvs5465
Copy link
Contributor

@mvs5465 mvs5465 commented Jan 21, 2017

Added snmpv2c functionality to rsu endpoint. Web GUI now has SNMP component that takes IP address and OID numeric path and submits them to /rsuHeartbeat endpoint, and appends response (if successful) to the div.

<div class="container">
<a class="navbar-brand" href="#">BSM ODE</a>
</div>
<div class="container"> <a class="navbar-brand" href="#">BSM ODE</a> </div>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this still an appropriate title since we are doing more than BSMs? Let's come up with a more generic title like "ODE Demo Console" or something in those lines.

@@ -62,6 +62,11 @@
<artifactId>junit</artifactId>
<scope>test</scope>
</dependency>
<dependency>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this dependency be added to jpo-ode-svcs project since other projects don't need it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, will make these changes

@hmusavi hmusavi merged commit bdc6739 into usdot-jpo-ode:feature/ode-128_rsuEndpoint Jan 21, 2017
iyourshaw pushed a commit to iyourshaw/jpo-ode that referenced this pull request Nov 25, 2022
…nment

C++ Debugging Support in Dev Container
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.

3 participants