refactor: introduce `FindTriggers` query, remove `FindTriggerById` - Githubissues

hyperledger-iroha / iroha

Iroha - A simple, enterprise-grade decentralized ledger

https://wiki.hyperledger.org/display/iroha

Apache License 2.0

438 stars 280 forks source link

refactor: introduce `FindTriggers` query, remove `FindTriggerById` #5040

Closed DCNick3 closed 1 month ago

DCNick3 commented 1 month ago

Context

Part of #4933.

Closes #5037.

Solution

Introduce FindTriggers query that can be used to retrieve all active triggers with their original wasm code and such
Remove FindTriggerById and replace its usages by FindTriggers
Leave FindActiveTriggerIds be until we have query projections

Migration Guide

FindTriggerById:

client
  .query(FindTriggers::new())
  .filter_with(|trigger| trigger.id.eq(needle_trigger_id))
  .execute_single()?

Checklist

[x] I've read CONTRIBUTING.md.
[ ] All review comments have been resolved.
[x] All CI checks pass.