-
Notifications
You must be signed in to change notification settings - Fork 10
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
Relay Requests Encoding and Tests #17
Conversation
8491c97
to
be1221e
Compare
be1221e
to
d60702d
Compare
Sources/NostrSDK/Filter.swift
Outdated
|
||
import Foundation | ||
|
||
struct Filter { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm hoping that we might be able to have this raw filter private to the library, and expose a filter type to the client app that uses native types, such as a NostrKind
enum for the kinds
part of the filter and Keypair
types for the pubkeys
. It could be converted the raw type when making requests, but that can come later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think we should for this right now? I think when we start setting up the public API we should definitely do that but not sure if we need to now. What do you think?
* Implement RelayRequest with Tests * remove self. and use new if let syntax * remove unnecessary codable implementation * use full request for name * use XCTUnwrap * test JSON equality protocol extension
#15