Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Add docs for notifying transactions via plugin#22097

Merged
lijunwangs merged 5 commits intosolana-labs:masterfrom
lijunwangs:add_docs_for_transactions_via_plugin
Dec 30, 2021
Merged

Add docs for notifying transactions via plugin#22097
lijunwangs merged 5 commits intosolana-labs:masterfrom
lijunwangs:add_docs_for_transactions_via_plugin

Conversation

@lijunwangs
Copy link
Copy Markdown
Contributor

@lijunwangs lijunwangs commented Dec 23, 2021

Problem

Summary of Changes

Add docs for notifying transactions via plugin
Fixes #

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 24, 2021

Codecov Report

Merging #22097 (e1f43de) into master (ddde356) will decrease coverage by 0.0%.
The diff coverage is 74.0%.

❗ Current head e1f43de differs from pull request most recent head 12950a9. Consider uploading reports for the commit 12950a9 to get more accurate results

@@            Coverage Diff            @@
##           master   #22097     +/-   ##
=========================================
- Coverage    81.3%    81.3%   -0.1%     
=========================================
  Files         518      518             
  Lines      145734   145764     +30     
=========================================
+ Hits       118554   118568     +14     
- Misses      27180    27196     +16     

Comment thread docs/src/developing/plugins/accountsdb_plugin.md Outdated
Comment thread docs/src/developing/plugins/accountsdb_plugin.md Outdated
Comment thread docs/src/developing/plugins/accountsdb_plugin.md Outdated
Comment thread docs/src/developing/plugins/accountsdb_plugin.md Outdated
Comment thread docs/src/developing/plugins/accountsdb_plugin.md Outdated
Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
@lijunwangs lijunwangs merged commit 135af08 into solana-labs:master Dec 30, 2021
mergify Bot pushed a commit that referenced this pull request Dec 30, 2021
* Added documentations for streaming transactions via plugin

* Updated comments for transaction info

* Updated doc on transaction format

* Removed a white space

* Apply suggestions from code review from Tyera

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
(cherry picked from commit 135af08)
mergify Bot added a commit that referenced this pull request Dec 30, 2021
* Added documentations for streaming transactions via plugin

* Updated comments for transaction info

* Updated doc on transaction format

* Removed a white space

* Apply suggestions from code review from Tyera

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
(cherry picked from commit 135af08)

Co-authored-by: Lijun Wang <83639177+lijunwangs@users.noreply.github.com>
@brooksprumo brooksprumo mentioned this pull request Jan 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants