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 JSON-RPC to v0.8.0 #630

Draft
wants to merge 44 commits into
base: main
Choose a base branch
from
Draft

Update JSON-RPC to v0.8.0 #630

wants to merge 44 commits into from

Conversation

FabijanC
Copy link
Contributor

@FabijanC FabijanC commented Oct 21, 2024

Usage related changes

Development related changes

Checklist:

  • Remove RPC v0.7.1 files and change RPC_SPEC_VERSION constant
  • Checked out the contribution guidelines
  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/clippy_check.sh
  • No unused dependencies - ./scripts/check_unused_deps.sh
  • No spelling errors - ./scripts/check_spelling.sh
  • Performed code self-review
  • Rebased to the latest commit of the target branch (or merged it into my branch)
    • Once you make the PR reviewable, please avoid force-pushing
  • Updated the docs if needed - ./website/README.md
  • Linked the issues resolvable by this PR - linking info
  • Updated the tests if needed; all passing - execution info

FabijanC and others added 7 commits October 7, 2024 18:07
* Change L1 hash repr: bytes array and string -> H256
* Move and rename tx status struct; add failure_reason

* Rename InsufficientMaxFee to InsufficientResourcesForValidate

* Improve panicking_contract: compile with 2.8.0, add method for distant panic
marioiordanov and others added 22 commits October 23, 2024 18:13
* SimulateTransactions integration tests (#628)

* remove handle_transaction_result, refactor errors

* integration tests

* added new error type

* typo

* fixed unit tests

* fixed integration test

* remove fully qualified names in errors file

* changed test name

* reuse variable

* replace declare_deploy_v1 with declare_v3_deploy_v3

* add simulate transaction validation check

* add functionality

* refactor if else with match 

* remove trait implementation

* changed error variant

* commment

* resolve compilation errors

* fix integration tests

* clippy

* removed let [skip ci]

* removed duplicated method [skip ci]

* remove duplicated test [skip ci]

* clippy

* clippy [skip ci]
* functions for compiling sierra to casm

* removed calculate_casm_hash function

* added getCompiledCasm endpoint

* change used function for compiling FlattenedSierraClass to Casm

* integration tests [skip ci]

* Reverted changes

* clippy

* added check in test

* change check in test
* Simplify event receiving via new util function
* Make aborting a BackgroundDevnet method

* Fix restarting behavior; check notifiability only if appropriate request

* Improve fn matches
* Rename block -> block_id

* Add missing BlockNotFound test

* Expand invalid block ID testing; move to separate file

* Add CallOnPending error

* Add receive_block in tests
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.

JSON-RPC API 0.8.0 adaptation
2 participants