-
Notifications
You must be signed in to change notification settings - Fork 8
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
Provide a simple way for potential physical nodes to mark themself on the map #131
Comments
The main technical limitation on this is how we store active
|
So what your saying is that this nodeshot replacement isn't going to be able to replicate basic nodeshot functionality any time soon? |
No, I'm explaining the technical reason that it doesn't. It's a I've outlined the problem and the solution. It's theoretically Uncrimethink |
It seems to me that basing the database around a users supposen IPv6 address is asking for trouble, and really limits NodeAtlas to specifically CJDNS/IPv6 nodes that already exist. Is there some major reason why NodeAtlas didn't start with internal unique user ids for each node right off the bat? Also, I'd love to go fork myself & fix this, but sadly there aren't enough hours in the day for me to do that currently. It just seems to me that this project has had a bad case of mission creep going on, as it originally started out as a generic replacement for Nodeshot, then it was supposed to be usable for any mapping application, and now here we are with something that does neither. |
Just as a follow up, it appears adding fun features like RSS is a higher priority than straightening out basic features like this based on the recent commit history? |
@danry25 @thefinn93 Perhaps you could contribute code instead of demands. |
@dansup this is basic functionality we're asking for here, not some one off feature that will be rarely used. |
@dansup These are fundamental changes to the structure of the whole thing, and neither of us speak go. You know this, stop trying to provoke us. |
I should mention two things: First, this is in the works (like a real way). |
Cannot require things like a cjdns IP, must be very clear that it is for potential nodes, and the display mechanism for showing these must be clearly different from the one showing Internet/UDP peers (currently called VPSs I believe)
The text was updated successfully, but these errors were encountered: