Skip to content
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

Action queue improvements: Improve failure reason storage #444

Closed
Tracked by #451
fordN opened this issue Jul 29, 2022 · 0 comments · Fixed by #454
Closed
Tracked by #451

Action queue improvements: Improve failure reason storage #444

fordN opened this issue Jul 29, 2022 · 0 comments · Fixed by #454
Assignees
Labels
enhancement New feature or request indexer-common

Comments

@fordN
Copy link
Contributor

fordN commented Jul 29, 2022

Currently the failure reason column gets populated by the error log message if a transaction fails. These log messages were designed to be logged to stdout not to be stored in a DB column. We should update the failure reason stringsso they are short, to the point, well formatted failureReason strings ready to be stored in the DB.

@fordN fordN added enhancement New feature or request indexer-common labels Jul 29, 2022
@fordN fordN added this to Indexer Jul 29, 2022
@fordN fordN moved this to 🗃️ Inbox in Indexer Jul 29, 2022
@fordN fordN moved this from 🗃️ Inbox to 📒 Todo in Indexer Jul 29, 2022
@hopeyen hopeyen moved this from 📒 Todo to 🏗️ In Progress in Indexer Aug 8, 2022
@hopeyen hopeyen moved this from 🏗️ In Progress to 🚩 In Review in Indexer Aug 11, 2022
@fordN fordN moved this from 🚩 In Review to ✅ Approved in Indexer Aug 18, 2022
Repository owner moved this from ✅ Approved to ❌ Closed in Indexer Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request indexer-common
Projects
Status: Closed
Development

Successfully merging a pull request may close this issue.

2 participants