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 Request: traceroute for webgui #179

Closed
cracky22 opened this issue Mar 30, 2024 · 6 comments · Fixed by #211
Closed

Feature Request: traceroute for webgui #179

cracky22 opened this issue Mar 30, 2024 · 6 comments · Fixed by #211
Labels
enhancement New feature or request parity Feature is in other app (iOS, Android, etc)

Comments

@cracky22
Copy link

The option to traceroute in the peer list would be great :)

@cracky22
Copy link
Author

cracky22 commented May 11, 2024

@caveman99 @sachaw @geeksville What do you think about the idea?

@sachaw
Copy link
Member

sachaw commented May 12, 2024

I agree, it is certainly a useful feature to have in the web UI. Feel free to have a go at implementing it, I provide guidance if required.

@cracky22
Copy link
Author

I agree, it is certainly a useful feature to have in the web UI. Feel free to have a go at implementing it, I provide guidance if required.

Hello Sachaw, I just had to smile a little :) I am a student (16 years old) and am graduating from secondary school this year (I come from Germany, Munich) but I would like to try to incorporate it when I get the chance / time. Do I need certain knowledge? I could offer html/js/css as well as python :)
thank you also for your support

@sachaw
Copy link
Member

sachaw commented May 15, 2024

That's great. Prerequisite knowledge isn't strictly require. Just some familiarity with JavaScript should suffice.
It's work looking through the codebase to wrap your head around what's going on. If you make any changes, feel free to open a PR and I can give feedback/assistance as required

fifieldt added a commit to fifieldt/meshtastic-web that referenced this issue May 26, 2024
As noted in meshtastic#179, it would be great to be able to
send traceroutes from the web application.

This patch adds a button to the messages page that sends a
traceroute to the selected node.
fifieldt added a commit to fifieldt/meshtastic-web that referenced this issue May 26, 2024
As noted in meshtastic#179, it would be great to be able to
send traceroutes from the web application.

This patch adds a button to the messages page that sends a
traceroute to the selected node.
@fifieldt
Copy link
Collaborator

Hi @sachaw , @cracky22 -- I've hacked together the start of some basic functionality over here:

#211

I probably won't be able to progress that much more before I go on holiday, so if anyone wants to take it over and hack on it over the next couple weeks please do feel free!

@fifieldt
Copy link
Collaborator

I've been using the code in #211 for the past couple weeks and asked some people in the local community to test as well. Result is: it works reliably :)

I don't have the skill to design a great UI integration, so hoping someone might be able to help get this over the line.

@Hunter275 Hunter275 added the enhancement New feature or request label Jun 15, 2024
@Hunter275 Hunter275 linked a pull request Jun 16, 2024 that will close this issue
@Hunter275 Hunter275 added the parity Feature is in other app (iOS, Android, etc) label Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request parity Feature is in other app (iOS, Android, etc)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants