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.

Sign up for GitHub

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

Jump to bottom

refactor: render all operations in Simple Transaction Viewer #509

Merged
merged 11 commits into from
Sep 13, 2024
Merged

refactor: render all operations in Simple Transaction Viewer #509

merged 11 commits into from
Sep 13, 2024

Conversation

arthurgeron
Copy link
Contributor

Closes #491

  • [Refactor]: Operations are now mapped, as well as their properties and behavior, instead of repetitive if/else statements
  • [Refactor]: Mixed components now exported to their own files and folders for better readability and maintainability
  • [Feature]: All operations are now rendered in Simple Transaction View (i.e. LOG, LOG_DATA, PANIC, REVERT, RETURN, RETURN_DATA and SCRIPT_RESULT)
  • [Chore]: Updated Browsers list (e.g. npx update-browserslist-db@latest)

@arthurgeron arthurgeron changed the title Ag/feat/tx operations feat: tx operations Sep 13, 2024
@arthurgeron arthurgeron changed the title feat: tx operations feat: render all operations in Simple Transaction Viewer Sep 13, 2024
@arthurgeron arthurgeron marked this pull request as ready for review September 13, 2024 19:15
@arthurgeron arthurgeron changed the title feat: render all operations in Simple Transaction Viewer refactor: render all operations in Simple Transaction Viewer Sep 13, 2024
rodrigobranas
rodrigobranas previously approved these changes Sep 13, 2024
View reviewed changes
@arthurgeron arthurgeron enabled auto-merge (squash) September 13, 2024 20:07
@arthurgeron arthurgeron merged commit 5992396 into main Sep 13, 2024
7 of 8 checks passed
@arthurgeron arthurgeron deleted the ag/feat/tx-operations branch September 13, 2024 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodrigobranas rodrigobranas rodrigobranas approved these changes

@luizstacio luizstacio Awaiting requested review from luizstacio luizstacio is a code owner

@pedronauck pedronauck Awaiting requested review from pedronauck pedronauck is a code owner

@LuizAsFight LuizAsFight Awaiting requested review from LuizAsFight LuizAsFight is a code owner

@helciofranco helciofranco Awaiting requested review from helciofranco helciofranco is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

show receipts on root level in Transaction - Operations section
2 participants
@arthurgeron @rodrigobranas