Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Apply RFRs in PR #300

Co-authored-by: Derek Pierre <derek.pierre@gmail.com>
  • Loading branch information
KPrasch and derekpierre authored Aug 15, 2024
1 parent 1e396c6 commit e60ab6d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions scripts/manage_subscription.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ def add_encryptors(account, network, domain, ritual_id, encryptors):
coordinator = registry.get_contract(contract_name="Coordinator", domain=domain)
ritual = coordinator.rituals(ritual_id)
access_controller = Contract(ritual.access_controller) # uses polygonscan API
if account != ritual.authority:
if account.address != ritual.authority:
raise ValueError(f"Only the authority ({ritual.authority}) can authorize encryptors.")

transactor = Transactor(account=account)
Expand All @@ -135,9 +135,9 @@ def remove_encryptors(account, network, domain, ritual_id, encryptors):
# lookup the access controller + authority for the ritual
coordinator = registry.get_contract(contract_name="Coordinator", domain=domain)
ritual = coordinator.rituals(ritual_id)
access_controller = Contract(ritual.access_controller) # uses polygonscan API
access_controller = Contract(ritual.accessController) # uses polygonscan API

if account != ritual.authority:
if account.address != ritual.authority:
raise ValueError(f"Only the authority ({ritual.authority}) can remove encryptors.")

click.echo(
Expand Down

0 comments on commit e60ab6d

Please sign in to comment.