Skip to content

lark: Refactor update authority tx to set adapter on initalize

tamsin woo requested to merge lark-update-listener into master

Local Checklist

  • Read our Contributing guide (for newcomers)
  • Tests written and passing locally?
  • Code style checked?
  • QA-ed locally?
  • Rebased with master branch?
  • Configuration updated (if needed)?
  • Documentation updated (if needed)?

What does this Merge Request do?

This is an attempt to tighten up the UpdateAuthority transaction interface by reducing the amount of data that needs to be passed in the Result (Success/Failure) monad. Since we'll be passing the adapter through the entire transaction anyway, it seems better to set it once, up front, and avoid passing it through step #calls.

The new approach is also in line with how event_stream is handled.

Merge request reports