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

Add get-predeployed-accounts function #125

Merged
merged 11 commits into from
Jul 6, 2022
Merged

Conversation

qperrot
Copy link
Contributor

@qperrot qperrot commented Jun 21, 2022

Usage related changes

  • Implementation of get-predeployed-accounts function, which return an array of objects, containing the data of all pre-deployed accounts (which are: address, initial_balance, private and public keys).
    • A clientside function for accessing the /predeployed_accounts endpoint on Devnet.

Development related changes

  • Upgrade to devnet v0.2.5
  • Update TOKEN_ADDRESS used in test
  • Fix formatting

Checklist:

  • I have formatted the code
  • I have performed a self-review of the code
  • I have rebased to the base branch
  • I have documented the changes
  • I have updated the tests
  • I have created a PR to the plugin branch of starknet-hardhat-example.

@qperrot qperrot requested a review from FabijanC June 21, 2022 14:35
src/devnet-utils.ts Outdated Show resolved Hide resolved
@qperrot qperrot requested a review from FabijanC June 22, 2022 09:49
scripts/test.sh Outdated Show resolved Hide resolved
@qperrot qperrot requested a review from FabijanC June 22, 2022 11:32
@FabijanC
Copy link
Collaborator

FabijanC commented Jun 22, 2022

This PR is blocked by this one: 0xSpaceShard/starknet-devnet#138
After that PR is merged, a new devnet version should be released (and plugin configured to use it in its tests).

config.json Outdated Show resolved Hide resolved
@FabijanC FabijanC changed the title implementation of get-predeployed-accounts function Add get-predeployed-accounts function Jul 6, 2022
@FabijanC FabijanC merged commit 913d616 into master Jul 6, 2022
@FabijanC FabijanC deleted the get-predeployed-accounts branch July 6, 2022 14:57
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.

None yet

3 participants