Skip to content

[CHANGE] Revert transaction & moved Market Price Update to Task#306

Merged
Geuthur merged 2 commits into
masterfrom
price-update
Jul 5, 2026
Merged

[CHANGE] Revert transaction & moved Market Price Update to Task#306
Geuthur merged 2 commits into
masterfrom
price-update

Conversation

@Geuthur

@Geuthur Geuthur commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Description

Changed

  • Revert transaction & moved Market Price Update to Task

Type of Changes

Please select the type of changes made in this pull request:

  • Bug Fix (non-breaking change which fixes an Issue)
  • New Feature (non-breaking change)
  • Other (please specify):

Checklist

Please ensure the following before submitting your pull request:

  • I have read the contributing guidelines.
  • My code follows the code style of this project.
  • I have performed a self-review of my code.
  • I have commented my code where necessary.
  • All new and existing tests passed.

@codecov

codecov Bot commented Jul 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
ledger/models/characteraudit.py 0.00% 2 Missing ⚠️
ledger/tasks.py 66.67% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #306      +/-   ##
==========================================
- Coverage   65.64%   65.52%   -0.12%     
==========================================
  Files          52       52              
  Lines        4362     4364       +2     
  Branches      444      444              
==========================================
- Hits         2863     2859       -4     
- Misses       1372     1378       +6     
  Partials      127      127              
Files with missing lines Coverage Δ
ledger/managers/character_mining_manager.py 60.19% <100.00%> (-1.85%) ⬇️
ledger/tasks.py 60.94% <66.67%> (-0.11%) ⬇️
ledger/models/characteraudit.py 83.90% <0.00%> (-2.01%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Geuthur Geuthur merged commit 66127e2 into master Jul 5, 2026
28 of 29 checks passed
@Geuthur Geuthur deleted the price-update branch July 5, 2026 20:15
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.

1 participant