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]: Nodes quick options #307

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

Conversation

KomelT
Copy link
Contributor

@KomelT KomelT commented Sep 18, 2024

Similar features are in mobile apps.
When you are on the "Nodes" page, you can click on the node name and a popup pops out.
For now, the popup contains Traceroute and Request position buttons.

68747470733a2f2f73342e657a6769662e636f6d2f746d702f657a6769662d342d653430386632303963662e676966

@Hunter275
Copy link
Member

@KomelT

The image you linked 404s

image

@KomelT
Copy link
Contributor Author

KomelT commented Sep 18, 2024

@Hunter275 it is just gif of a feature. Now it should work.

@Hunter275 Hunter275 added enhancement New feature or request major change This is a major change labels Nov 10, 2024
@KomelT KomelT marked this pull request as ready for review January 23, 2025 22:24
@KomelT
Copy link
Contributor Author

KomelT commented Jan 23, 2025

This should be good to go if there aren't any more wishes.

EDIT:
I forgot now there are 2 way traceroutes. I will try to add also that.

@KomelT
Copy link
Contributor Author

KomelT commented Jan 23, 2025

@Hunter275 now it should be ready
image

@KomelT
Copy link
Contributor Author

KomelT commented Jan 24, 2025

This is latest.
Screenshot_20250124_121025

@KomelT
Copy link
Contributor Author

KomelT commented Jan 30, 2025

Screenshots of "Node Details"
Screenshot_20250130_180659
Screenshot_20250130_180712

timestamp: number;
}

const getTimeAgo = (timestamp: number): string => {
Copy link
Collaborator

@danditomaso danditomaso Jan 30, 2025

Choose a reason for hiding this comment

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

I'd prefer we avoid using setIntervals to manually calculate time. There is a Browser API Intl which offers a built in api to calculate elapsed time, here is an article with some code you could re-use:
https://dev.to/madsstoumann/showing-time-ago-in-a-social-feed-using-intlrelativetimeformat-5ceo

@danditomaso
Copy link
Collaborator

I think there is also some issues with the time ago that is being calculated (any nodes that show "days ago")

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request major change This is a major change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants