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

Fixes for v0.7 EntryPoint #37

Merged
merged 9 commits into from
Dec 13, 2024
Merged

Fixes for v0.7 EntryPoint #37

merged 9 commits into from
Dec 13, 2024

Conversation

mmontour1306
Copy link

fix: Remove paymasterAndData and fix signature packing for v0.7 EntryPoint.

Changes to be committed:
modified: packages/snap/src/keyring.ts
modified: packages/snap/src/utils/ecdsa.ts

…Point.

 Changes to be committed:
	modified:   packages/snap/src/keyring.ts
	modified:   packages/snap/src/utils/ecdsa.ts
 Changes to be committed:
	modified:   packages/snap/src/keyring.ts
 Changes to be committed:
	modified:   packages/site/src/config/snap.ts
	modified:   packages/site/src/utils/snap.ts
…sfer.

 Changes to be committed:
	modified:   packages/snap/src/keyring.ts
 Changes to be committed:
	modified:   packages/site/src/pages/index.tsx
 Changes to be committed:
	modified:   packages/site/src/config/snap.ts
	modified:   packages/site/src/pages/index.tsx
	modified:   packages/snap/src/keyring.ts
	modified:   packages/snap/src/utils/ecdsa.ts
 Changes to be committed:
	modified:   packages/snap/src/keyring.ts
 Changes to be committed:
	modified:   package.json
	modified:   packages/site/package.json
	modified:   packages/snap/package.json
	modified:   packages/snap/snap.manifest.json
 Changes to be committed:
	modified:   packages/snap/snap.manifest.json
	modified:   packages/snap/src/keyring.ts
@mmontour1306 mmontour1306 marked this pull request as ready for review December 12, 2024 00:57
@mmontour1306 mmontour1306 requested a review from wsdt December 12, 2024 00:57
@mmontour1306
Copy link
Author

The "use Boba as paymaster" option is not currently supported but I think this works apart from that, now including the deployment of the account contract on the first transaction (translating the initCode field into factory/factoryData).

@wsdt wsdt merged commit 59ede67 into main Dec 13, 2024
8 checks passed
@wsdt wsdt deleted the mm/support-v7 branch December 13, 2024 09:30
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.

3 participants