Skip to content

Update Random#783

Open
N-010 wants to merge 3 commits intoqubic:developfrom
N-010:feature/2026-03-05-update-contract-RANDOM
Open

Update Random#783
N-010 wants to merge 3 commits intoqubic:developfrom
N-010:feature/2026-03-05-update-contract-RANDOM

Conversation

@N-010
Copy link
Contributor

@N-010 N-010 commented Mar 5, 2026

Updaet from CFB

@N-010 N-010 marked this pull request as ready for review March 5, 2026 15:10
Copy link
Collaborator

@Franziska-Mueller Franziska-Mueller left a comment

Choose a reason for hiding this comment

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

please merge develop and follow the new format for StateData and explicit read/write access via .get() / .mut()

N-010 added 2 commits March 12, 2026 14:11
…05-update-contract-RANDOM

# Conflicts:
#	src/contracts/Random.h
@N-010 N-010 requested a review from Franziska-Mueller March 12, 2026 11:37
@N-010
Copy link
Contributor Author

N-010 commented Mar 12, 2026

#783 (review)
Done


struct StateData
{
uint64 earnedAmount;
Copy link
Collaborator

Choose a reason for hiding this comment

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

you are not using the fee and these amounts anywhere in the code

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