Skip to content

[N4] Adapt neo-node to Neo 4.0.0#989

Open
ajara87 wants to merge 63 commits intoneo-project:masterfrom
ajara87:adapt-neo-node-neo-4
Open

[N4] Adapt neo-node to Neo 4.0.0#989
ajara87 wants to merge 63 commits intoneo-project:masterfrom
ajara87:adapt-neo-node-neo-4

Conversation

@ajara87
Copy link
Member

@ajara87 ajara87 commented Jan 28, 2026

Update the master branch to reflect the changes made in Neo 4.0.0

Related to #988

@github-actions github-actions bot added the N4 label Jan 28, 2026
@ajara87 ajara87 marked this pull request as draft January 28, 2026 23:49
@ajara87 ajara87 mentioned this pull request Jan 28, 2026
@shargon
Copy link
Member

shargon commented Feb 10, 2026

UT still fault

@ajara87
Copy link
Member Author

ajara87 commented Feb 10, 2026

UT still fault

Its a draft. Still working on it

@Wi1l-B0t
Copy link
Contributor

Wi1l-B0t commented Mar 9, 2026

This PR is too large, and it can only modify the parts that cause compilation errors.
Adaptation for new features can be added later.

@Wi1l-B0t
Copy link
Contributor

Any progress?

@ajara87
Copy link
Member Author

ajara87 commented Mar 19, 2026

Any progress?

working on It. Its just 8 tests to fix and it's ready

@ajara87 ajara87 marked this pull request as ready for review March 20, 2026 20:19
@ajara87 ajara87 requested review from Wi1l-B0t and shargon March 22, 2026 12:29
Comment on lines 81 to 82
if (_enabledTrackers.Contains("NEP-11"))
trackers.Add(new Nep11Tracker(_db, _maxResults, _shouldTrackHistory, neoSystem));
Copy link
Member

Choose a reason for hiding this comment

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

Why NEP11 is working and Nep17 removed?
I think both should be removed

Copy link
Member Author

Choose a reason for hiding this comment

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

I can remove it in a different PR. Nep17 had to remove them within this PR because its tests were failing and to avoid removing it they would have had to be adapted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants