Skip to content

Update predeclared OZ account class_hash #3236

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

Closed
kkawula opened this issue Apr 17, 2025 · 1 comment · Fixed by #3235
Closed

Update predeclared OZ account class_hash #3236

kkawula opened this issue Apr 17, 2025 · 1 comment · Fixed by #3235
Assignees
Labels

Comments

@kkawula
Copy link
Member

kkawula commented Apr 17, 2025

Current State

Current account doesn't allow to tract sierra gas

Objective

Update to preset 1.0.0 or higher https://docs.openzeppelin.com/contracts-cairo/1.0.0/presets
blocked - 0xSpaceShard/starknet-devnet#767

Additional Context

Blocked by devnet

@cptartur
Copy link
Member

@kkawula @ddoktorski verify if we really use these accounts in tests (it should mostly use predeclares already)

@cptartur cptartur moved this from TODO to In Progress in Starknet foundry May 14, 2025
github-merge-queue bot pushed a commit that referenced this issue May 15, 2025
<!-- Reference any GitHub issues resolved by this PR -->

Closes #3236

Updated OZ account class_hash to the latest declared. This change allows
charging fee using sierra gas.
- https://docs.openzeppelin.com/contracts-cairo/1.0.0/presets


## Introduced changes

<!-- A brief description of the changes -->

-

## Checklist

<!-- Make sure all of these are complete -->

- [x] Linked relevant issue
- [x] Updated relevant documentation
- [x] Added relevant tests
- [x] Performed self-review of the code
- [x] Added changes to `CHANGELOG.md`
@github-project-automation github-project-automation bot moved this from In Progress to Done in Starknet foundry May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants