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

fix: Include automatic deployment feature on executeTxn for get-starknet in release/2.3.0 #254

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

khanti42
Copy link
Collaborator

No description provided.

* feat: add webpack.config.dev.js for local serving of remoteEntry.js

* chore: lint and optional env variable for SNAP_ID

* fix: update get-starknet/webpack-dev-server dep to address security vulnerability

* fix: remoteEntry.js serving done on static file from dist/webpack folder

* chore: added .env.sample example file

* feat: optional PUBLIC_PATH env variable to produce local remoteEntry.js

* chore: updated README.md

* fix: env variable for local environment resolved at build time

* fix: check and deploy on executeTxn

* fix: isAccountDeployed requires publicKey

* feat: add waiting mode for create account

* feat: wait for tx in executTxn if account not deployed

* test: adapted executeTxn tests to account for accountDeployment feature

* style: prettier

* style: lint

* fix: waitForDeployment

* test: fix executeTxn

* fix: lint

* fix: add waitForTransaction to maximize chance of success

* fix: snap.manifest.json endowment:rpc

* fix: uses estimateFeeBulk directly

* style: remove console.log

* test: executeTxn tests with account deployed or not

* test: improve coverage and readability

* test: rollback to match MM style

---------

Co-authored-by: stanleyyuen <[email protected]>
@khanti42 khanti42 requested a review from a team as a code owner June 17, 2024 12:56
@khanti42 khanti42 requested review from Julink-eth and wantedsystem and removed request for a team June 17, 2024 12:56
@khanti42 khanti42 changed the title hotfix: Include automatic deployment feature on executeTxn for get-starknet in release/2.3.0 fix: Include automatic deployment feature on executeTxn for get-starknet in release/2.3.0 Jun 17, 2024
@stanleyyconsensys stanleyyconsensys merged commit 0af0afb into releases/2.3.0 Jun 18, 2024
6 of 7 checks passed
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.

2 participants