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

API redesign #254

Open
jiftechnify opened this issue Apr 3, 2024 · 2 comments
Open

API redesign #254

jiftechnify opened this issue Apr 3, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@jiftechnify
Copy link
Owner

  • More intuitive filter syntax
  • Provide intuitive way to fetch replaceable events
  • Provide various streaming styles (async iterables, event handlers, Rx observables)
  • "Partial application" of relays to fetch (and options?)

... any ideas?

@jiftechnify jiftechnify changed the title APi redesign API redesign Apr 3, 2024
@jiftechnify jiftechnify added the enhancement New feature or request label Apr 3, 2024
@dskvr
Copy link

dskvr commented Jun 26, 2024

More intuitive filter syntax

On this note, it would definitely be better if it accepted a standard filter object and then nostr-fetch validated/extracted the prerequisite properties from the filter internally, as opposed to splitting the filter into filter and time-range

@dskvr
Copy link

dskvr commented Jun 26, 2024

Another thing that comes to mind is that the relaysUrl argument is unintuitive/confusing/unclear when using an adapter with an existing pool, for example NDK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

2 participants