Joystream / Joystream/joystream

QN: Improve Metaprotocol transactions error handling

Open
#4,573 0 comments 0 reactions 1 assignee Claimed by @zeeshanakram3 View on GitHub
enhancement metaprotocol query-node
Dominant language
Rust
Stars
1.4k
Forks
116
PR merge metrics
No merged PRs in 30d

Description

As pointed out by [Lezsek](https://github.com/Joystream/joystream/pull/4517#discussion_r1086299357), we need improved & efficient implementation of error handling logic for metaprotocol txs which include features like `comments/reactions`, `videoCategory`, `channelpayment`, `apps` etc.

Further ...
> Going forward it would be good to make metaprotocol transaction errors just enum variants instead of storing the metaprotocol error message in the database, as this would eventually become a lot of wasted storage, and it's unproportionally cheap to spam QN database this way. The frontend could decide what kind of error message to display based on the error code/variant instead. Also it would be language-agnostic approach

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.