Skip to content

feat: process nanocontract transactions and nc_addresses#263

Merged
r4mmer merged 9 commits intomasterfrom
feat/ingest-nano-contract-tx
Jul 22, 2025
Merged

feat: process nanocontract transactions and nc_addresses#263
r4mmer merged 9 commits intomasterfrom
feat/ingest-nano-contract-tx

Conversation

@r4mmer
Copy link
Copy Markdown
Member

@r4mmer r4mmer commented Jul 14, 2025

Motivation

To maintain complete transaction history for wallets and addresses, we need to adapt our indexing service to handle these new transactions.

Acceptance Criteria

  • Receive nano header from fullnode events
  • Add the transaction to the nc_address history
  • Clean the transaction from the nc_address history when the tx is voided

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.

@r4mmer r4mmer self-assigned this Jul 14, 2025
@r4mmer r4mmer moved this from Todo to In Progress (WIP) in Hathor Network Jul 14, 2025
@r4mmer r4mmer moved this from In Progress (WIP) to In Progress (Done) in Hathor Network Jul 16, 2025
@r4mmer
Merge remote-tracking branch 'origin/master' into feat/ingest-nano-co…
40627c8 
…ntract-tx

* origin/master:
  chore: upgrade wallet-lib to v2.5.1 (#222)
andreabadesso
andreabadesso requested changes Jul 21, 2025
View reviewed changes
Comment thread packages/daemon/src/db/index.ts Outdated
Comment thread packages/daemon/src/services/index.ts Outdated
Comment thread packages/daemon/src/utils/wallet.ts Outdated
@github-project-automation github-project-automation Bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jul 21, 2025
pedroferreira1
pedroferreira1 reviewed Jul 22, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@pedroferreira1 pedroferreira1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's missing the seqnum handling

Comment thread packages/daemon/src/db/index.ts
Comment thread packages/daemon/src/services/index.ts Outdated
@r4mmer r4mmer moved this from In Review (WIP) to In Progress (Done) in Hathor Network Jul 22, 2025
@github-project-automation github-project-automation Bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jul 22, 2025
@r4mmer r4mmer merged commit a020c4a into master Jul 22, 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 Jul 22, 2025
@r4mmer r4mmer deleted the feat/ingest-nano-contract-tx branch July 22, 2025 19:57
r4mmer added a commit that referenced this pull request Jul 24, 2025
@r4mmer
Merge remote-tracking branch 'origin/master' into feat/nc-events-tests
9be6f6a 
* origin/master:
  chore: testnet hotel (#264)
  feat: process nanocontract transactions and nc_addresses (#263)
@r4mmer r4mmer moved this from Waiting to be deployed to Done in Hathor Network Aug 21, 2025
This was referenced Nov 17, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedroferreira1 pedroferreira1 pedroferreira1 approved these changes

@andreabadesso andreabadesso andreabadesso approved these changes

Assignees

@r4mmer r4mmer

Labels

None yet

Projects

Hathor Network
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@r4mmer @pedroferreira1 @andreabadesso