Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

Feat: update klay sdks #624

Merged
merged 10 commits into from
Jun 10, 2024
Merged

Conversation

Sotatek-TanHoang
Copy link
Contributor

@Sotatek-TanHoang Sotatek-TanHoang commented Jun 4, 2024

  1. Add specs for:
    - klay_getBlsInfor
    - klay_getActiveAddressFromRegistry
    - klay_getAllRecordsFromRegistry
    - klay_getTotalSupply
  2. Update category for klay_gasPrice, klay_gasPriceAt, eth_gasPrice
  3. Update description for:
    - klay_gasPrice
    - eth_gasPrice
    - klay_maxPriorityFeePerGas
    - eth_maxPriorityFeePerGas
    - klay_feeHistory
    - eth_feeHistory
  4. Update [Request > params > TracingOptions]

@Sotatek-TanHoang Sotatek-TanHoang changed the title Feat/update klay sdk Feat: update klay sdks Jun 4, 2024
Copy link
Contributor

@blukat29 blukat29 left a comment

Choose a reason for hiding this comment

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

Thanks!

@@ -15,7 +15,7 @@ repositories {
}

dependencies {
api "foundation.klaytn:web3rpc-java:v0.9.0"
api "foundation.klaytn:web3rpc-java:v1.0.1"
implementation "org.web3j:core:4.9.7"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The generated sdk have version 1.0.1 so i need to change build.gradle version of web3rpc-java to match the local build

@kjeom kjeom assigned kjeom and Sotatek-TanHoang and unassigned kjeom Jun 10, 2024
params:
type: array
items:
anyOf:
Copy link
Member

Choose a reason for hiding this comment

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

@Sotatek-TanHoang As I said, you should avoid anyOf type

params:
type: array
items:
anyOf:
Copy link
Member

Choose a reason for hiding this comment

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

Same comments. Please look at above

@kjeom kjeom merged commit a9066db into klaytn:dev Jun 10, 2024
2 checks passed
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.

3 participants