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

feat: 2.3.0 release cario 1 upgrade #177

Closed
wants to merge 12 commits into from
Closed

Conversation

stanleyyconsensys
Copy link
Collaborator

@stanleyyconsensys stanleyyconsensys commented Nov 17, 2023

  • add util methods:

    • findAddressIndex , to find address index from the keyDeriver, and return the address index and it's carioVersions
    • getBalance to call contract interface balanceOf
    • isUpgradeRequired to check the given address needed upgrade or not
    • getCorrectContractAddress to get user address by public key, prioritize return cario 1 address over cario 0
    • getAccContractAddressAndCallDataCairo0 to calculate cario 0 contract address by publicKey
    • getAccContractAddressAndCallDataCairo0 to calculate cario 0 contract address by publicKey
  • update util methods:

    • isAccountDeployed to call contract interface getVersion
    • getKeysFromAddress to adopt method findAddressIndex
    • add cario version 1 when init account instance
  • adopt util method:

    • consolidate account deploy checking to util method isAccountDeployed
    • consolidate get balance to util method getBalance
    • apply isUpgradeRequired to estimateFee, extractPrivateKey, extractPublicKey, sendTransaction, signMessage, verifySignedMessage , to block request if the account require upgrade
    • apply getCorrectContractAddress in recover account to return correct address
  • update unit test to split the test case into cario 1 vs cario 0 , and require upgrade vs not require upgrade

  • remove accountClassHash from network, and place by constant variable ACCOUNT_CLASS_HASH_V0 and ACCOUNT_CLASS_HASH_V1.

stanleyyconsensys and others added 8 commits October 4, 2023 18:50
* feat: change account contract to cairo 1

- update recover account to return correct address
- update create account to use isAccountAddressDeployed method
- update est gas fee to use new accountClassHash
- add upgradeRequired in to acc object
- fix test case

* fix: update incorrect name

---------

Co-authored-by: Amine Harty <[email protected]>
* feat: change account contract to cairo 1

- update recover account to return correct address
- update create account to use isAccountAddressDeployed method
- update est gas fee to use new accountClassHash
- add upgradeRequired in to acc object
- fix test case

* feat: add blocking on snap method

- add blocking on snap methods
- add test case

* fix: update incorrect name

* fix: lint style

* fix: bug in merge conflict

* fix: update yarn lock
Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.5.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.5)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [@adobe/css-tools](https://github.com/adobe/css-tools) from 4.2.0 to 4.3.1.
- [Changelog](https://github.com/adobe/css-tools/blob/main/History.md)
- [Commits](https://github.com/adobe/css-tools/commits)

---
updated-dependencies:
- dependency-name: "@adobe/css-tools"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [semver](https://github.com/npm/node-semver) from 7.3.8 to 7.5.2.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v7.3.8...v7.5.2)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@stanleyyconsensys stanleyyconsensys changed the title Releases/2.3.0 feat: 2.3.0 release cario 1 upgrade Nov 17, 2023
@stanleyyconsensys stanleyyconsensys self-assigned this Nov 17, 2023
Copy link

[consensys_starknet-snap-starknet-snap] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

90.5% 90.5% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Copy link

[consensys_starknet-snap-wallet-ui] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant