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

feat: compile time context provider #1570

Open
wants to merge 130 commits into
base: v1.0-dev
Choose a base branch
from
Open

feat: compile time context provider #1570

wants to merge 130 commits into from
+11,354 −1,092

Conversation

QuantumExplorer
Copy link
Contributor

Issue being fixed or feature implemented

What was done?

How Has This Been Tested?

Breaking Changes

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
  • I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

lklimek and others added 30 commits June 28, 2023 14:03
@lklimek
feat!(drive): add block_id_hash to proof
28ba89c
@lklimek
chore(dapi-grpc): format .proto code
d3ffb01
@lklimek
feat(proto): include quorum_type in Proof and chain_id in ResponseMet…
008d551 
…adata
@lklimek
feat: Drive Light Client with proof verification
7e1ffdd
@lklimek
chore: update to tenderdash 0.13.0-dev.1
bf20d9e
@lklimek
chore: code cleanup and docs
6db7fd5
@lklimek
fix(drive): invalid error message
70f11ec
@lklimek
test: tests working
2329759
@lklimek
test: uniffi proof test
7c2bdf6
@jawid-h
chore: no documents and identities by pkh
8121b8c
@jawid-h
chore: identities by pk_hash
9dc981c
@jawid-h
chore: get documents
df024b1
@lklimek
Merge remote-tracking branch 'origin/v0.25-dev' into feat/rs-drive-li…
f064493 
…ght-client
@lklimek
chore: fix platform.proto after merge
85e3fb2
@lklimek
feat(dapi-grpc): add serde support with serde feature
a42b5e9
@lklimek
test: drive-light-client test vectors support
01b8265
@lklimek
fix: dapi bindings generation broken
009737f
@lklimek
test(light-client): test get identities by hashes not found
76e18e2
@lklimek
feat(dapi-grpc): enable serde feature by default
5075014
@lklimek
test: move uniffi tests from src/ to tests/
6ecedb7
@lklimek
chore(dapi-grpc): simplify deserialization
df7f913
@lklimek
feat: json encoding for bindings
9cb7a0f
@lklimek
chore: rebuild light client bindings
fa4c475
@lklimek
Merge remote-tracking branch 'origin/v0.25-dev' into feat/rs-drive-li…
e4410b5 
…ght-client
@lklimek
chore: cargo update
734ee54
@lklimek
chore: fix deps after merge
9d88a30
@lklimek
chore: move around bindings code
24a6340
@lklimek
chore: typo
cdda96e
@lklimek
Merge remote-tracking branch 'origin/v0.25-dev' into feat/rs-drive-li…
82ef535 
…ght-client
@lklimek
chore: add uniffi feature to disable uniffi code (remove in future)
269145c
QuantumExplorer and others added 23 commits November 6, 2023 18:21
@QuantumExplorer
fix: fixing sdk merge (#1564)
6cad150
@QuantumExplorer
Rs sdk/merge 1.0 with new proto (#1543)
3282e5d
@lklimek
Merge remote-tracking branch 'origin/feat/rs-drive-light-client' into…
ecded07 
… feat/register
@lklimek
refactor(rs-dapi-client): define Mockable interface instead of relyin…
8a180b0 
…g on serde
@lklimek
refactor: replace MockRequest with Mockable trait
c54859e
@lklimek
chore: typo
9047cc3
@lklimek
test: update rs-sdk test vectors
cff5c43
@lklimek
fix: DumpData serialization is invalid
f647f5d
@lklimek
test: offline testing by deafult
456ae40
@lklimek
test: rs-sdk update test vectors
b5b9301
@lklimek
chore: typo
3ab5ad3
@lklimek
chore: self-review
dd3dce5
@QuantumExplorer
refactor(rs-sdk): introduce Mockable trait to simplify handling of st…
c74ef9a 
…reaming (#1565)
@QuantumExplorer
more work
be00570
@lklimek
refactor: rename QuorumInfoProvider to ContextProvider
b3dee98
@lklimek
chore: fix build issues
e05543a
@lklimek
refactor(dapi-client): don't require mutable self
b8de25b
@lklimek
feat(rs-sdk): data contract cache
05933ca
@lklimek
chore: fix some warnings
3616294
@lklimek
refactor: move code around
5778568
@QuantumExplorer
feat!: rs-sdk cache of data contracts (#1568)
a097cc3
@QuantumExplorer
small fixes
ee53660
@QuantumExplorer
refactor: change context provider to not use dyn
2249265
@QuantumExplorer QuantumExplorer changed the base branch from feat/register to v1.0-dev November 9, 2023 03:46
@QuantumExplorer QuantumExplorer changed the base branch from v1.0-dev to feat/register November 9, 2023 03:47
@QuantumExplorer QuantumExplorer changed the title Feat/compile time context provider feat: compile time context provider Nov 9, 2023
lklimek
lklimek reviewed Nov 9, 2023
View reviewed changes
Copy link
Contributor

@lklimek lklimek left a comment

Choose a reason for hiding this comment

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

The PR looks good, but it will not work through C bindings (I assume ContextProvider might be provided by external developer, from their language like C; in this case, we need dynamic context provider).

Base automatically changed from feat/register to v1.0-dev February 12, 2024 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lklimek lklimek lklimek left review comments

@shumkov shumkov Awaiting requested review from shumkov shumkov is a code owner

@antouhou antouhou Awaiting requested review from antouhou

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@QuantumExplorer @lklimek @jawid-h @shotonoff @fominok