Conversation
projectName, | ||
orgId | ||
} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is a bit confusing to request a specific content although we are asserting that there will be no transactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still working on meaningful tests, this is wip.
proxy/src/registry.rs
Outdated
}) | ||
}; | ||
|
||
self.transactions.insert(tx.id, tx.clone()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a good idea to implement a self.submit_tx
proxy function that calls self.client.submit_transaction
and inserts the tx in the cache. That way we don't ever forget to cache stuff.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately that is not doable at the moment as Message
in the registry is a trait with a static method. So we can't really pull the code out that constructs registry transactions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌱
Closes #244