-
Notifications
You must be signed in to change notification settings - Fork 172
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
[Chat] feature/#244-create thread #266
Conversation
llbartekll
commented
Jun 10, 2022
•
edited
Loading
edited
- allows to create a thread for two peers
- still using old encryption policy
…V2 into #chat/accept-invite # Conflicts: # Sources/Chat/Chat.swift # Sources/Chat/Engine.swift # Sources/Chat/NetworkingInteractor.swift # Sources/Chat/Types/ChatRequest.swift # Sources/Chat/Types/Invite.swift # Tests/ChatTests/EndToEndTests.swift # Tests/ChatTests/Mocks/NetworkingInteractorMock.swift # Tests/ChatTests/RegistryManagerTests.swift
let registryManager: RegistryManager | ||
let engine: Engine | ||
let registryService: RegistryService | ||
let invitationHandlingService: InvitationHandlingService |
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 like ***Service naming, let's use more often
@@ -5,12 +5,23 @@ import WalletConnectRelay | |||
import WalletConnectUtils | |||
|
|||
protocol NetworkInteracting { | |||
func subscribe(topic: String) async throws | |||
var requestPublisher: AnyPublisher<RequestSubscriptionPayload, Never> {get} |
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.
run swiftlint --fix
please
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.
done
…V2 into #chat/accept-invite # Conflicts: # Sources/Chat/Chat.swift # Sources/Chat/Engine.swift # Sources/Chat/NetworkingInteractor.swift # Sources/Chat/Types/ChatRequest.swift # Sources/Chat/Types/Invite.swift # Sources/WalletConnectKMS/Codec/ChaChaPolyCodec.swift # Tests/ChatTests/EndToEndTests.swift
* Add registry manager * Update api methods * update resolve * Add description to api methods * update invite method * remove teardown * make registryStore private * update new thread e2e test * Add invite service * savepoint * Update invite service * Add invitation handling * test - decrypts an invite * savepoint * Accept and create a Thread on Invitee * savepoint * add json rpc history * savepoint * remove ChatRequest type * create a Thread on both peers * clean up * swiftlint fix