Skip to content

Fix broken Solidity snippet (callback example did not compile)#1129

Open
Meesut0 wants to merge 1 commit intobase:masterfrom
Meesut0:parameters
Open

Fix broken Solidity snippet (callback example did not compile)#1129
Meesut0 wants to merge 1 commit intobase:masterfrom
Meesut0:parameters

Conversation

@Meesut0
Copy link

@Meesut0 Meesut0 commented Feb 13, 2026

What changed? Why?
The intermediate callback snippet (lines 211–218) did not compile due to incorrect variable names used inside the function body.

The function parameters were defined as _nonce and _rngList, but the snippet referenced nonce and rngList, resulting in multiple DeclarationError: Undeclared identifier errors.

How has it been tested?
Steps taken:

  • Copied the snippet into Remix IDE
  • Confirmed compilation failed
  • Replaced rngList > _rngList
  • Replaced nonce > _nonce
  • Recompiled successfully

The snippet is now consistent with the final contract example and compiles correctly.

@cb-heimdall
Copy link
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants