Skip to content

Commit

Permalink
Merge pull request #9 from xiaolou86/main
Browse files Browse the repository at this point in the history
Fix typos
  • Loading branch information
eloi010 authored Nov 10, 2023
2 parents 335f1e0 + 5acc3ea commit 1da64f7
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ As of June 2023, the current average gas cost for deploying or using the differe
| StaticOpenfortFactory | Deploy factory (containing StaticOpenfortAccount's implementation) | 1 | ~2,500,000 |
| StaticOpenfortAccount | Create a new static account using the `createAccountWithNonce()` of the factory | indefinite | ~150,000 |
| StaticOpenfortAccount | Updating the EntryPoint address using `updateEntryPoint()` | indefinite | ~1,500 |
| StaticOpenfortAccount | Transfering the ownership using `transferOwnership()` | indefinite | ~25,000 |
| StaticOpenfortAccount | Transferring the ownership using `transferOwnership()` | indefinite | ~25,000 |

### Upgradeable Accounts
| Smart Contract | Description | # of deployments per game/ecosystem | Avg gas cost |
Expand Down
2 changes: 1 addition & 1 deletion contracts/core/BaseOpenfortAccount.sol
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ abstract contract BaseOpenfortAccount is
* @param _validAfter - this session key is valid only after this timestamp.
* @param _validUntil - this session key is valid only up to this timestamp.
* @notice using this function will automatically set the sessionkey as a
* master session key because no further restricion was set.
* master session key because no further restriction was set.
* @notice default limit set to 100.
*/
function registerSessionKey(address _key, uint48 _validAfter, uint48 _validUntil) public {
Expand Down
2 changes: 1 addition & 1 deletion contracts/core/eip6551/EIP6551OpenfortAccount.sol
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,7 @@ contract EIP6551OpenfortAccount is
* @param _validAfter - this session key is valid only after this timestamp.
* @param _validUntil - this session key is valid only up to this timestamp.
* @notice using this function will automatically set the sessionkey as a
* master session key because no further restricion was set.
* master session key because no further restriction was set.
* @notice default limit set to 100.
*/
function registerSessionKey(address _key, uint48 _validAfter, uint48 _validUntil) public {
Expand Down
2 changes: 1 addition & 1 deletion test/foundry/core/eip6551/EIP6551OpenfortBenchmark.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -524,7 +524,7 @@ contract EIP6551OpenfortBenchmark is Test {

/*
* Test transferOwnership() function using upgradeable account that have EIP6551 accounts.
* Scenario: a complex account changes the ownership; all NFTs are managable by the new owner
* Scenario: a complex account changes the ownership; all NFTs are manageable by the new owner
*/
function test8TransferOwner4337Complex() public {
// The EOA is the owner of the Upgradeable account
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1975,7 +1975,7 @@ contract RecoverableOpenfortAccountTest is Test {
*/

/*
* Check the correct functionallity of startRecovery()
* Check the correct functionality of startRecovery()
*/
function testStartRecovery() public {
RecoverableOpenfortAccount recoverableOpenfortAccount = RecoverableOpenfortAccount(payable(account));
Expand Down
4 changes: 2 additions & 2 deletions test/foundry/paymaster/OpenfortPaymasterV2Test.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -1572,7 +1572,7 @@ contract OpenfortPaymasterV2Test is Test {
function testAcceptOwnershipComplex() public {
assertEq(openfortPaymaster.owner(), paymasterAdmin);

// Play arround with deposits
// Play around with deposits
assertEq(openfortPaymaster.getDeposit(), 50 ether);
assertEq(openfortPaymaster.getDepositFor(paymasterAdmin), 50 ether);
assertEq(openfortPaymaster.getDepositFor(factoryAdmin), 0 ether);
Expand All @@ -1589,7 +1589,7 @@ contract OpenfortPaymasterV2Test is Test {
assertEq(openfortPaymaster.getDeposit(), 53 ether);
assertEq(openfortPaymaster.getDepositFor(paymasterAdmin), 50 ether);
assertEq(openfortPaymaster.getDepositFor(factoryAdmin), 3 ether);
// Play arround with deposits
// Play around with deposits

vm.prank(factoryAdmin);
openfortPaymaster.acceptOwnership();
Expand Down

0 comments on commit 1da64f7

Please sign in to comment.