Skip to content
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.

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

5% block reward code to AVN foundation wallet #87

Merged
merged 5 commits into from
Sep 20, 2022

Conversation

alamshafil
Copy link
Contributor

5% block reward code to AVN foundation wallet

@alamshafil alamshafil added the enhancement New feature or request label Jul 13, 2022
@alamshafil alamshafil added this to the v4.1.0 milestone Jul 13, 2022
@alamshafil
Copy link
Contributor Author

Important to have #71 to scan code. Rebasing against updated 4.1 branch.

@alamshafil alamshafil self-assigned this Jul 13, 2022
src/blockencodings.cpp Outdated Show resolved Hide resolved
src/chainparams.cpp Outdated Show resolved Hide resolved
src/chainparams.cpp Outdated Show resolved Hide resolved
src/consensus/tx_verify.cpp Outdated Show resolved Hide resolved
src/chainparams.cpp Outdated Show resolved Hide resolved
@alamshafil alamshafil added consensus Consensus change do not merge yet Do not merge PR labels Jul 13, 2022
@alamshafil
Copy link
Contributor Author

Since this PR deals with a consensus change, I believe having #72 is important to check that nothing broke or regressed with this PR.

@alamshafil
Copy link
Contributor Author

Working as expected on regtest.

FounderPayment::FillFounderPayment -- Founder payment 3125000000 to n1BurnXXXXXXXXXXXXXXXXXXXXXXU1qejP

   {
      "value": 31.25000000,
      "n": 1,
      "scriptPubKey": {
        "asm": "OP_DUP OP_HASH160 d7c8944771bbfe427418f27320e72a1322faf134 OP_EQUALVERIFY OP_CHECKSIG",
        "hex": "76a914d7c8944771bbfe427418f27320e72a1322faf13488ac",
        "reqSigs": 1,
        "type": "pubkeyhash",
        "addresses": [
          "n1BurnXXXXXXXXXXXXXXXXXXXXXXU1qejP"
        ]
      }
    }

@alamshafil alamshafil force-pushed the 4.1-foundation-rewards branch 2 times, most recently from 8e5ccd7 to 98449bf Compare September 9, 2022 00:38
@alamshafil alamshafil removed the do not merge yet Do not merge PR label Sep 9, 2022
This was referenced Sep 12, 2022
@alamshafil alamshafil mentioned this pull request Sep 19, 2022
@alamshafil alamshafil merged commit 0137fee into 4.1 Sep 20, 2022
@alamshafil alamshafil deleted the 4.1-foundation-rewards branch September 20, 2022 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus Consensus change enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants