Skip to content

Fix TRON processing by not requiring running a custom node #101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
alexqrid wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix TRON processing by not requiring running a custom node #101

alexqrid wants to merge 2 commits into from

Conversation

@alexqrid
Copy link
Collaborator

@alexqrid alexqrid commented Jul 31, 2024

As the PR in tronprotocol and the ugly behaviour in the node still exists:

  • Refactoring the code in order to be able to use official node without patch
    Also:
  • Separated JSON-RPC and REST endpoints
  • Fixed resource delegation processing
  • Added tests via the new flow from the Add a new system for testing modules #100

@alexqrid alexqrid self-assigned this Jul 31, 2024
@alexqrid alexqrid requested a review from Har01d as a code owner July 31, 2024 21:55
@alexqrid alexqrid mentioned this pull request Jul 31, 2024
Open
@alexqrid alexqrid changed the title get rid off patched node and refactor delegation processing get rid off patched TRON node and refactor delegation processing Aug 5, 2024
@Har01d Har01d changed the title get rid off patched TRON node and refactor delegation processing Fix TRON processing by not requiring running a custom node Sep 24, 2024
@Har01d Har01d self-assigned this Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Har01d Har01d Awaiting requested review from Har01d Har01d is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@Har01d Har01d

@alexqrid alexqrid

Labels

Refactor blockchain

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexqrid @Har01d