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

Update struct prefixes to Ui #10874

Merged
merged 2 commits into from
Jul 1, 2020
Merged

Conversation

CriesofCarrots
Copy link
Contributor

Problem

A lot of things are called Rpc when they don't necessarily have anything to do with rpc messaging. We recently came up with a new Ui prefix to replace Rpc in account-decoder crate.

Summary of Changes

  • Use Ui prefix in transaction-status structs

@CriesofCarrots CriesofCarrots requested a review from mvines July 1, 2020 18:42
Copy link
Member

@mvines mvines left a comment

Choose a reason for hiding this comment

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

Cool. It feels better to not see "Rpc" things in places like ledger/src/blockstore.rs

@CriesofCarrots CriesofCarrots added the automerge Merge this Pull Request automatically once CI passes label Jul 1, 2020
@codecov
Copy link

codecov bot commented Jul 1, 2020

Codecov Report

Merging #10874 into master will decrease coverage by 0.0%.
The diff coverage is 64.2%.

@@            Coverage Diff            @@
##           master   #10874     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%     
=========================================
  Files         308      308             
  Lines       71464    71464             
=========================================
- Hits        58545    58543      -2     
- Misses      12919    12921      +2     

@mergify mergify bot merged commit 2669ccb into solana-labs:master Jul 1, 2020
CriesofCarrots added a commit to CriesofCarrots/solana that referenced this pull request Jul 7, 2020
* Update comments

* Use Ui prefix
CriesofCarrots added a commit to CriesofCarrots/solana that referenced this pull request Jul 7, 2020
* Update comments

* Use Ui prefix
CriesofCarrots added a commit that referenced this pull request Jul 7, 2020
* Revert "Rpc: add filter to getProgramAccounts (#10888) (#10932)"

This reverts commit 9311a6e.

* Add jsonParsed option for EncodedTransactions; add memo parser (#10711)

* Add jsonParsed option for EncodedTransactions; add memo parser

* Use kebab case for program names

* Add account-key parsing

* Add parse test

* Update transaction encoding docs (#10833)

* Add account-decoder utilities (#10846)

* Fix comment and make less pub

* Add account-decoder crate and use to decode vote and system (nonce) accounts

* Update docs

* Rename RpcAccount struct

* s/Rpc/Display

* Call it jsonParsed and update docs

* Revert "s/Rpc/Display"

This reverts commit 6e7149f.

* s/Rpc/Ui

* Add tests

* Ui more things

* Comments

* Update struct prefixes to Ui (#10874)

* Update comments

* Use Ui prefix

* Rpc: add filter to getProgramAccounts (#10888)

* Add RpcFilterType, and implement CompareBytes for getProgramAccounts

* Accept bytes in bs58

* Rename to memcmp

* Add Memcmp optional encoding field

* Add dataSize filter

* Update docs

* Clippy

* Simplify tests that don't need to test account contents; add multiple-filter tests
@CriesofCarrots CriesofCarrots deleted the ui-tx-status branch July 24, 2020 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants