bcnmy / scw-contracts

SCW contracts for Biconomy Smart Account
MIT License
115 stars 82 forks source link

Multi owned ecdsa remediations #189

Closed filmakarov closed 8 months ago

filmakarov commented 8 months ago

Summary

Issues remediations according to the audit

Related Issue: #347

Change Type

Checklist

linear[bot] commented 8 months ago

SMA-347 MultiOwner ECDSA Kawach

livingrockrises commented 8 months ago

Approved

filmakarov commented 8 months ago

Waiting for Auditor's comments

Aboudjem commented 8 months ago

LGTM just need this to be updated:

string public constant NAME = "ECDSA Ownership Registry Module";

livingrockrises commented 8 months ago

LGTM just need this to be updated:

string public constant NAME = "ECDSA Ownership Registry Module";

Is this resolved? @filmakarov

livingrockrises commented 8 months ago

Waiting for Auditor's comments

also resolve this if done. ready to merge from my side

filmakarov commented 8 months ago

Fixed the name. Auditors approval pending