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

Pass each relay's latest as its since #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alopatindev
Copy link

This updates nostr-tools because there's no subscribeManyMap in the older version.

trackRelays should've been enabled, otherwise seenOn is empty.

I would actually personally prefer to pass not the since itself but something like since - tenMins (or even since - hour), just to make sure we don't miss some "delayed" event sent from machine with poor time synchronization (or even perfect synchronization, but sent over a poor network just a second before the last event; or even broadcasted from some other relay using some cron script, etc.).

@alopatindev alopatindev force-pushed the pass-relays-latest branch from 5a325a3 to e49411d Compare July 7, 2024 15:58
@franzaps
Copy link
Owner

franzaps commented Jul 8, 2024

Thanks. Are you using zapthreads for anything in particular or just want to improve it?

@alopatindev
Copy link
Author

I'm about to integrate it to my website as soon as I'll finish some features. Basically I want to make it a bit faster and add some filtering.

@franzaps
Copy link
Owner

franzaps commented Jul 9, 2024

That's cool, share your site if you want

@alopatindev
Copy link
Author

alopatindev commented Jul 9, 2024

Sure, here it is, zapthreads is not integrated though yet

https://codonaft.com

@alopatindev
Copy link
Author

Ok, finally running it 🚀

Here's how it looks like and what features are available now:
https://github.com/codonaft/zapthreads-codonaft#readme

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.

2 participants