Skip to content

fix: add nano contracts enabled#296

Merged
andreabadesso merged 1 commit intomasterfrom
fix/add-nano-contracts-enabled
Sep 24, 2025
Merged

fix: add nano contracts enabled#296
andreabadesso merged 1 commit intomasterfrom
fix/add-nano-contracts-enabled

Conversation

@andreabadesso
Copy link
Copy Markdown
Collaborator

@andreabadesso andreabadesso commented Sep 23, 2025

Motivation

  • The fullnode API now returns a nano_contracts_enabled field that indicates whether nano contracts are enabled on the network. The wallet-service needs to expose this information through its version endpoint.

Acceptance Criteria

  • Version API returns nanoContractsEnabled field
  • Field is validated as required boolean when receiving data from fullnode
  • Field defaults to false when missing from older fullnode versions
  • Database storage and retrieval of the field works correctly

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged
  • Make sure either the unit tests and/or the QA tests are capable of testing the new features
  • 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 force-pushed the fix/add-nano-contracts-enabled branch 2 times, most recently from 168e3d3 to 3e65e1f Compare September 23, 2025 23:09
@andreabadesso andreabadesso self-assigned this Sep 24, 2025
@andreabadesso andreabadesso added the enhancement New feature or request label Sep 24, 2025
@andreabadesso andreabadesso moved this to In Progress (WIP) in Hathor Network Sep 24, 2025
@andreabadesso andreabadesso force-pushed the fix/add-nano-contracts-enabled branch 7 times, most recently from 52ceda7 to fe47b42 Compare September 24, 2025 17:36
@andreabadesso andreabadesso force-pushed the fix/add-nano-contracts-enabled branch from fe47b42 to 57961a4 Compare September 24, 2025 17:41
@github-project-automation github-project-automation Bot moved this from In Progress (WIP) to In Review (WIP) in Hathor Network Sep 24, 2025
@andreabadesso andreabadesso merged commit 8326c96 into master Sep 24, 2025
1 check passed
@github-project-automation github-project-automation Bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Sep 24, 2025
This was referenced Nov 17, 2025
@andreabadesso andreabadesso moved this from Waiting to be deployed to Done in Hathor Network Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants