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

feature: add ability to send NMEA 2000 out with the Data Fiddler #1755

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sbender9
Copy link
Member

No description provided.

@sbender9 sbender9 requested a review from tkurki June 13, 2024 13:51
Copy link
Member

@tkurki tkurki left a comment

Choose a reason for hiding this comment

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

I think this is a cool idea, but I think it needs a little more work to be usable for end users.

Could we update the textarea description with something that would describe this option and mention the specific formats that can be used? We have a bit of documentation for them at /documentation/develop/plugins/deltas.html?highlight=NMEA%202000%20json#sending-nmea-2000-data-from-a-plugin so we could link there.

The N2K button is now next to Beatify JSON button, when its function is closer to send to server

image

We have nmea2000OutAvailable, the button could be shown only when that is available? And the button could be enabled only when the input is valid for sending n2k.

We are spelling out NMEA 2000 in the textarea description, let's use it here also. How about naming the button Send as PGN to server's NMEA2000 connection? Too long maybe...It is always a bit hard to understand what is out and in....the textarea is under Input and then you have Output on the right, but the input here is for sending data OUT from the server, not into the server for processing....

@@ -114,6 +115,13 @@ module.exports = function (app) {
return
}

if (sendToN2K && type != 'n2k-json' && type != 'n2k') {
res.status(400).json({
error: 'PLease enter NMEA 2000 json format or Actisense format'
Copy link
Member

Choose a reason for hiding this comment

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

lowercase L

@tkurki
Copy link
Member

tkurki commented Jun 24, 2024

Do you think the ability to send arbitrary PGNs should be available only for admin users?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants