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

RFC(0045): Client Side Block Filter Protocol #375

Merged
merged 9 commits into from
Nov 8, 2022

Conversation

quake
Copy link
Member

@quake quake commented Sep 26, 2022

@quake quake requested a review from a team as a code owner September 26, 2022 03:00
@quake quake requested a review from doitian September 26, 2022 03:00
@doitian doitian linked an issue Oct 11, 2022 that may be closed by this pull request
5 tasks
bors bot added a commit to nervosnetwork/ckb that referenced this pull request Oct 18, 2022
3631: feat: add light client support to ckb full node r=zhangsoledad a=quake

### What problem does this PR solve?

add light client support to ckb full node

### What is changed and how it works?

related rfcs:
[RFC(0043): CKB softfork activation](nervosnetwork/rfcs#349)
[RFC: CKB Light Client Protocol](nervosnetwork/rfcs#370)
[RFC: Client Side Block Filter Protocol](nervosnetwork/rfcs#375)

### Check List <!--REMOVE the items that are not applicable-->

Tests <!-- At least one of them must be included. -->

- Unit test
- Integration test

### Release note <!-- Choose from None, Title Only and Note. Bugfixes or new features need a release note. -->

```release-note
Note: Add a note under the PR title in the release note.
```



Co-authored-by: Boyu Yang <yangby@cryptape.com>
Co-authored-by: quake <quake.wang@gmail.com>
Co-authored-by: Linfeng Qian <thewawar@gmail.com>
Co-authored-by: Quake Wang <quake.wang@gmail.com>
doitian
doitian previously approved these changes Oct 19, 2022
@doitian doitian mentioned this pull request Oct 19, 2022
14 tasks
@doitian
Copy link
Member

doitian commented Oct 19, 2022

Use RFC Number 45

Copy link
Member

@janx janx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also add a new row to the RFC table in README.

rfcs/0000-client-block-filter/0000-client-block-filter.md Outdated Show resolved Hide resolved
rfcs/0000-client-block-filter/0000-client-block-filter.md Outdated Show resolved Hide resolved
rfcs/0000-client-block-filter/0000-client-block-filter.md Outdated Show resolved Hide resolved
Co-authored-by: busyforking <5958+janx@users.noreply.github.com>
Co-authored-by: zhangsoledad <787953403@qq.com>
@quake quake changed the title RFC: Client Side Block Filter Protocol RFC(0045): Client Side Block Filter Protocol Oct 26, 2022
@quake quake force-pushed the quake/client-side-filter branch from 9425762 to b6eefcf Compare October 26, 2022 02:37
@quake quake force-pushed the quake/client-side-filter branch from b6eefcf to 5e0625a Compare October 26, 2022 02:40
doitian
doitian previously approved these changes Oct 26, 2022
Copy link
Member

@janx janx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a row to the RFC list on README.

Co-authored-by: zhangsoledad <787953403@qq.com>
janx and others added 2 commits November 8, 2022 12:54
Co-authored-by: Boyu Yang <yangby@cryptape.com>
Co-authored-by: Boyu Yang <yangby@cryptape.com>
@janx janx merged commit af5c8f3 into nervosnetwork:master Nov 8, 2022
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.

todo list of release 0.105.0
5 participants