Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: William Richter <88400283+WRRicht3r@users.noreply.github.com>
  • Loading branch information
outoftardis and WRRicht3r authored Jun 1, 2022
1 parent 8194758 commit b2de434
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/guide/advanced/triggers.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ determines the scope of the trigger. <!-- Check -->

### `Trigger.metadata`

The `Metadata` is the same kind of `Metadata` that can be attached to
This `Metadata` is the same kind of `Metadata` that can be attached to
accounts, domains, assets, or transactions.

### `Trigger.action`
Expand Down Expand Up @@ -94,8 +94,8 @@ A technical account is the account that would (in theory) be responsible
for the execution environment and be the authority for `Instruction`
execution.

For now, you can leave this to be the account that registered the trigger.
If you followed the tutorial, this is `alice@wonderland`. However, later on
For now, you can leave this as the account that registered the trigger.
If you have been following the tutorial, this is `alice@wonderland`. However, later on
we will show you why you'd want to create a brand new account for those
purposes.

Expand Down Expand Up @@ -140,7 +140,7 @@ got registered, an asset got transferred, the Queen of Hearts decided to
burn all of her assets.

These types of events account for the vast majority of triggers in
Ethereum, and were the first to be implemented. As of today we only support
Ethereum, and were the first to be implemented. As of today, we only support
un-scoped system-wide triggers with no permission validation. Work is
ongoing to make the triggers safer and more reliable, but the process is
time-consuming and work-intensive. <!-- Q: still true? -->
Expand Down

0 comments on commit b2de434

Please sign in to comment.