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
Support RPC 0.2.1 + drop deploy transaction #212
Conversation
Codecov ReportBase: 70.69% // Head: 70.19% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## milestone/0.4.0 #212 +/- ##
===================================================
- Coverage 70.69% 70.19% -0.51%
===================================================
Files 56 58 +2
Lines 1573 1778 +205
Branches 114 117 +3
===================================================
+ Hits 1112 1248 +136
- Misses 430 492 +62
- Partials 31 38 +7
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't reviewed the javademo yet. Will include it in another review.
...in/com/swmansion/starknet/data/serializers/JsonRpcTransactionReceiptPolymorphicSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/Contracts.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/Contracts.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Javademo review.
3321399
to
aad62f8
Compare
...in/com/swmansion/starknet/data/serializers/JsonRpcTransactionReceiptPolymorphicSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/ContractClassGatewaySerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/transactions/TransactionReceipt.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/transactions/Transaction.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/exceptions/RequestFailedException.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/rpc/JsonRpcProvider.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/TransactionPolymorphicSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/TransactionPolymorphicSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/Responses.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/transactions/Transaction.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/types/transactions/Transaction.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/gateway/GatewayProvider.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/rpc/JsonRpcProvider.kt
Outdated
Show resolved
Hide resolved
d4e4939
to
54b8c4d
Compare
…Index, pendingTransactions and addDeployAccountTransaction
54b8c4d
to
2c8c200
Compare
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/HexToIntSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/exceptions/RequestFailedException.kt
Show resolved
Hide resolved
2530658
to
4631f70
Compare
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/HexToIntSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/TransactionPolymorphicSerializer.kt
Outdated
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/data/serializers/TransactionPolymorphicSerializer.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/exceptions/RequestFailedException.kt
Show resolved
Hide resolved
lib/src/main/kotlin/com/swmansion/starknet/provider/gateway/GatewayProvider.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see my comment about payloads in estimateFee.
Describe your changes
This PR introduces support for RPC 0.2.1 spec and drops support for deploy transaction (gateway + rpc).
Linked issues
Closes #146
Closes #148
Closes #190
Closes #184
Closes #202
Breaking changes
This issue contains breaking changes
RPC 0.1.0 support is dropped
provider.deployContract is dropped