Skip to content

fix: added missing tx proposal delete schema#969

Merged
pedroferreira1 merged 3 commits intomasterfrom
fix/missing-txProposalDelete-schema
Nov 19, 2025
Merged

fix: added missing tx proposal delete schema#969
pedroferreira1 merged 3 commits intomasterfrom
fix/missing-txProposalDelete-schema

Conversation

@andreabadesso
Copy link
Copy Markdown
Contributor

@andreabadesso andreabadesso commented Nov 18, 2025

Acceptance Criteria

  • We must update the schema for deleting a tx proposal

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@andreabadesso andreabadesso self-assigned this Nov 18, 2025
@andreabadesso andreabadesso added the bug Something isn't working label Nov 18, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.75%. Comparing base (7d5a382) to head (78e367a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #969      +/-   ##
==========================================
+ Coverage   84.68%   84.75%   +0.06%     
==========================================
  Files         112      112              
  Lines        8145     8146       +1     
  Branches     1762     1753       -9     
==========================================
+ Hits         6898     6904       +6     
+ Misses       1219     1214       -5     
  Partials       28       28              

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

luislhl
luislhl previously approved these changes Nov 18, 2025
pedroferreira1
pedroferreira1 previously approved these changes Nov 18, 2025
@github-project-automation github-project-automation bot moved this from In Review (Done) to In Review (WIP) in Hathor Network Nov 19, 2025
@pedroferreira1 pedroferreira1 merged commit ecbad3f into master Nov 19, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Nov 19, 2025
@pedroferreira1 pedroferreira1 deleted the fix/missing-txProposalDelete-schema branch November 19, 2025 16:07
@andreabadesso andreabadesso moved this from Waiting to be deployed to Done in Hathor Network Nov 24, 2025
r4mmer added a commit that referenced this pull request Dec 11, 2025
…-policy

* origin/master: (31 commits)
  feat: fee token creation (#858)
  fix: checkAddressMine was crashing when called with an empty array (#977)
  feat: the nano amount field should accept 0
  Merge pull request #975 from HathorNetwork/chore/bump-v2.11.0
  feat: add nano execution logs API (#973)
  Refactor: Fullnode facade to Typescript (#972)
  Merge pull request #970 from HathorNetwork/chore/bump-v2.10.0
  fix: added missing tx proposal delete schema (#969)
  feat: add missing api calls for graphviz and nano (#967)
  Merge pull request #965 from HathorNetwork/feat/complete-token-info-object
  chore: bump package to v2.9.1 (#962)
  feat: catch and store async promise error (#960)
  chore: bump wallet lib to v2.9.0 (#958)
  tests: use core v0.67.0 docker image for the integration tests (#956)
  tests: update integration tests blueprints for the new core sdk (#954)
  fix: added a minimal accessData so methods that need storage don't crash (#955)
  feat: fee header (#951)
  feat: added read-only start (#950)
  fix: dont add metadata changed for voided txs (#948)
  test: Wallet Service infrastructure (#909)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants