Skip to content

Commit

Permalink
chore: removed and updated comments
Browse files Browse the repository at this point in the history
  • Loading branch information
sifnoc committed Nov 9, 2023
1 parent 5a9120f commit c575a29
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 13 deletions.
11 changes: 8 additions & 3 deletions backend/examples/summa_solvency_flow.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,14 @@ async fn main() -> Result<(), Box<dyn Error>> {
//
// The signer instance would be shared with `address_ownership` and `round` instances
//
// Using AddressInput::Address to directly provide the summa_contract's address.
// For deployed contracts, if the address is stored in a config file,
// you can alternatively use AddressInput::Path to specify the file's path.
// Using `AddressInput::Address`` to directly provide the summa_contract's address.
//
// If the address of a deployed contract is stored in a configuration file,
// you can use `AddressInput::Path` to provide the path to that file.
//
// For example, if the contract address is in "backend/src/contracts/deployments.json" located
// you would use `AddressInput::Path` as follows:`AddressInput::Path("backend/src/contracts/deployments.json".to_string())`.
//
let signer = SummaSigner::new(
"0xac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80",
anvil.endpoint().as_str(),
Expand Down
11 changes: 1 addition & 10 deletions contracts/src/Summa.sol
Original file line number Diff line number Diff line change
Expand Up @@ -84,16 +84,7 @@ contract Summa is Ownable {
constructor(IVerifier _inclusionVerifier) {
inclusionVerifier = _inclusionVerifier;
}

/*
// While this duplicate method might elevate deployment costs, the trade-off is a reduction in usage and maintenance costs.
function submitProofOfAddressOwnership(
AddressOwnershipProof memory _addressOwnershipProof
) public onlyOwner {
...
}
*/


/**
* @dev Submit an optimistic proof of multiple address ownership for a CEX. The proof is subject to an off-chain verification as it's not feasible to verify the signatures of non-EVM chains in an Ethereum smart contract.
* @param _addressOwnershipProofs The list of address ownership proofs
Expand Down

0 comments on commit c575a29

Please sign in to comment.