Skip to content

feat: Combined layout for transactions #3304

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
Open
infrmtcs wants to merge 1 commit into
base: dat/migrate-block-transactions
Choose a base branch
from
Open

feat: Combined layout for transactions #3304

infrmtcs wants to merge 1 commit into from

Conversation

@infrmtcs
Copy link
Contributor

@infrmtcs infrmtcs commented Nov 25, 2025

No description provided.

@infrmtcs infrmtcs added the disable-deploy-test We don't want to run deploy tests with this PR because it might affect our development environment. label Nov 25, 2025
This was referenced Nov 25, 2025
Closed
Open
@codecov
Copy link

codecov bot commented Nov 28, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 70.00000% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.67%. Comparing base (2a8d36d) to head (6166e7d).

Files with missing lines Patch % Lines
core/accessors.go 64.58% 7 Missing and 10 partials ⚠️
blockchain/blockchain.go 80.00% 2 Missing and 2 partials ⚠️
Additional details and impacted files 
@@                        Coverage Diff                         @@
##           dat/migrate-block-transactions    #3304      +/-   ##
==================================================================
- Coverage                           75.68%   75.67%   -0.01%     
==================================================================
  Files                                 358      358              
  Lines                               33627    33629       +2     
==================================================================
- Hits                                25449    25448       -1     
- Misses                               6361     6362       +1     
- Partials                             1817     1819       +2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

rodrodros
rodrodros reviewed Dec 1, 2025
View reviewed changes
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch 2 times, most recently from 2dfe1e7 to dd31f0b Compare December 2, 2025 14:20
@infrmtcs infrmtcs force-pushed the dat/accessors branch 2 times, most recently from d40c6dc to ef27574 Compare December 3, 2025 05:22
Base automatically changed from dat/accessors to main December 3, 2025 11:19
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch 2 times, most recently from c016cdb to 27e2db3 Compare December 8, 2025 07:01
@rodrodros rodrodros requested a review from EgeCaner December 8, 2025 11:21
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch 2 times, most recently from 5051fb3 to 0834f3d Compare December 10, 2025 08:46
@infrmtcs infrmtcs changed the base branch from main to dat/closer December 26, 2025 11:07
@infrmtcs infrmtcs changed the base branch from dat/closer to dat/block-transactions December 26, 2025 11:13
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch from f208739 to d8b4ae2 Compare December 28, 2025 14:40
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch 2 times, most recently from 34f4feb to 9e15688 Compare December 28, 2025 16:58
@infrmtcs infrmtcs force-pushed the dat/block-transactions branch 3 times, most recently from 8367591 to 492d696 Compare December 30, 2025 08:48
@infrmtcs infrmtcs changed the base branch from dat/block-transactions to dat/migrate-block-transactions December 30, 2025 08:52
@infrmtcs infrmtcs force-pushed the dat/migrate-block-transactions branch from b650e4f to 60b4f1a Compare December 30, 2025 15:26
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch from 3e6a8b8 to 2673d1f Compare January 6, 2026 09:29
@infrmtcs infrmtcs force-pushed the dat/migrate-block-transactions branch 3 times, most recently from 597f206 to a9bb926 Compare January 7, 2026 12:38
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch from 2673d1f to 4b7b39e Compare January 7, 2026 12:38
@infrmtcs infrmtcs force-pushed the dat/migrate-block-transactions branch from a9bb926 to 7aa89b8 Compare January 7, 2026 12:53
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch from 4b7b39e to 5e6f0d7 Compare January 7, 2026 12:54
@infrmtcs infrmtcs force-pushed the dat/migrate-block-transactions branch from 7aa89b8 to 856e10a Compare January 15, 2026 09:25
@infrmtcs infrmtcs force-pushed the dat/bench/combined branch 2 times, most recently from 4e8336c to 128e665 Compare January 15, 2026 14:36
@infrmtcs infrmtcs force-pushed the dat/migrate-block-transactions branch from 856e10a to c0505ed Compare January 15, 2026 14:36
@infrmtcs infrmtcs force-pushed the dat/migrate-block-transactions branch from c0505ed to 2a8d36d Compare January 16, 2026 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rodrodros rodrodros rodrodros left review comments

@MaksymMalicki MaksymMalicki MaksymMalicki approved these changes

@EgeCaner EgeCaner Awaiting requested review from EgeCaner

Assignees

No one assigned

Labels

disable-deploy-test We don't want to run deploy tests with this PR because it might affect our development environment.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@infrmtcs @rodrodros @MaksymMalicki