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.

Sign up for GitHub

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

Jump to bottom

Rejection reasons #70

Merged
merged 28 commits into from
Feb 9, 2022
Merged

Rejection reasons #70

merged 28 commits into from
Feb 9, 2022

Conversation

llbartekll
Copy link
Contributor

close #69

@llbartekll
Add Automatic connection handler
24849d5
@llbartekll
add socket connection handler to dispatcher
eb2b227
@llbartekll
remove socket connection handling from wallet connect client
6dcf93b
@llbartekll
add manual and automatic socket connection type
17753fe
@llbartekll
Add manual socket connection handler
2179835
@llbartekll
Fix Tests
fdc11fc
@llbartekll
move network monitoring to automatic socket connection handler
5b53903
@llbartekll
Add manual socket connection tests
5eaeb15
@llbartekll
add automatic socket connection handler tests
6c0f6e7
@llbartekll
Update tests for automatic socket connection
935a2e2
@llbartekll
extract class into a file
1d0c517
@llbartekll
fix build
b1707eb
@llbartekll
invalidate task
a6d0ed4
@llbartekll
fixes
0f4e3c2
@llbartekll
fix race condition
1e6ed7f 
remove force unwrap in dapp
@llbartekll
segregate websocket session interfaces
022a7f5
@llbartekll
revert client interface changes
8a0f2c2
@llbartekll
print data on socket message
b9ac587
@llbartekll
Add initializers or waku and wc client
19f41a5
@llbartekll
update initialisers, add docs
e6d0bf8
@llbartekll
rename Relayer
ada029a
@llbartekll
add example with injectable relayer in dapp
1e52194
@llbartekll
remove controller flag, fix tests
969b919
@llbartekll
remove controller flag in remaining places
32285a1
@llbartekll
Restrict rejection reason to possible CAIP-25 codes
72d395e
@llbartekll llbartekll marked this pull request as draft February 9, 2022 09:45
@llbartekll llbartekll marked this pull request as ready for review February 9, 2022 09:48
@llbartekll llbartekll self-assigned this Feb 9, 2022
@sekimondre sekimondre merged commit 07cac7b into develop Feb 9, 2022
@sekimondre sekimondre deleted the rejection-reasons branch February 9, 2022 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sekimondre sekimondre sekimondre approved these changes

Assignees

@llbartekll llbartekll

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@llbartekll @sekimondre