-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
switch more EVM components to use sqlutil.DataStore #12856
Conversation
d4bac24
to
4d6101d
Compare
4d6101d
to
acf68a5
Compare
acf68a5
to
be06252
Compare
be06252
to
614868f
Compare
614868f
to
d930925
Compare
d930925
to
7d0dcc7
Compare
7d0dcc7
to
7bd98ce
Compare
7bd98ce
to
8fe6ea6
Compare
8fe6ea6
to
3d838a0
Compare
Quality Gate passedIssues Measures |
@@ -565,55 +565,53 @@ func (lsn *listenerV2) enqueueForceFulfillment( | |||
} | |||
|
|||
// fulfill the request through the VRF owner | |||
err = lsn.q.Transaction(func(tx pg.Queryer) error { |
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.
Weird this was a transaction in the first place 🤔
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.
Yeah, maybe it was just nudged to a confused state from edits over time 🤷
https://smartcontract-it.atlassian.net/browse/BCF-2978
This was extracted from #12456