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

fix: handle all tx were omiga_inscription situation #1578

Closed

Conversation

zmcNotafraid
Copy link
Collaborator

If a block's txs were all omiga_inscriptioon, we can uniq them to fast build_udt

github-actions bot and others added 4 commits January 16, 2024 12:44
* feat: adjust halving miner reward (nervosnetwork#1505)

* fix: xudt mainnet hash_type was data1

Signed-off-by: Miles Zhang <[email protected]>

---------

Signed-off-by: Miles Zhang <[email protected]>
Co-authored-by: Miles Zhang <[email protected]>
Co-authored-by: NanZhang <[email protected]>
* feat: adjust halving miner reward (nervosnetwork#1505)

* fix: xudt mainnet hash_type was data1

Signed-off-by: Miles Zhang <[email protected]>

* chore: optimize cell_type query

---------

Signed-off-by: Miles Zhang <[email protected]>
Co-authored-by: Miles Zhang <[email protected]>
Co-authored-by: NanZhang <[email protected]>
@zmcNotafraid zmcNotafraid changed the base branch from develop to master January 18, 2024 02:49
@zmcNotafraid zmcNotafraid changed the base branch from master to develop January 18, 2024 02:49
@codecov-commenter
Copy link

Codecov Report

Attention: 222 lines in your changes are missing coverage. Please review.

Comparison is base (bb447e4) 66.37% compared to head (b76c730) 67.50%.
Report is 202 commits behind head on develop.

❗ Current head b76c730 differs from pull request most recent head db6a2b9. Consider uploading reports for the commit db6a2b9 to get more accurate results

Files Patch % Lines
app/workers/charts/ckb_hodl_waves_statistic.rb 0.00% 45 Missing ⚠️
app/models/ckb_sync/new_node_data_processor.rb 83.61% 29 Missing ⚠️
.../controllers/api/v2/ckb_transactions_controller.rb 67.56% 24 Missing ⚠️
app/workers/token_transfer_detect_worker.rb 10.52% 17 Missing ⚠️
app/workers/contract_statistic_worker.rb 0.00% 15 Missing ⚠️
...p/serializers/omiga_inscription_info_serializer.rb 0.00% 14 Missing ⚠️
app/utils/ckb_utils.rb 84.09% 14 Missing ⚠️
app/controllers/api/v2/scripts_controller.rb 69.76% 13 Missing ⚠️
...ontrollers/api/v1/omiga_inscriptions_controller.rb 80.48% 8 Missing ⚠️
app/models/concerns/display_cells.rb 92.68% 6 Missing ⚠️
... and 14 more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1578      +/-   ##
===========================================
+ Coverage    66.37%   67.50%   +1.12%     
===========================================
  Files          275      289      +14     
  Lines         7397     7946     +549     
===========================================
+ Hits          4910     5364     +454     
- Misses        2487     2582      +95     

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

@Keith-CY Keith-CY changed the base branch from develop to master January 18, 2024 03:05
@Keith-CY
Copy link
Collaborator

Is this one ready to merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants