Skip to content

Feat/mini game reward#49

Open
viettu-bic wants to merge 18 commits into
masterfrom
feat/mini-game-reward
Open

Feat/mini game reward#49
viettu-bic wants to merge 18 commits into
masterfrom
feat/mini-game-reward

Conversation

@viettu-bic

Copy link
Copy Markdown
Contributor

No description provided.

@viettu-bic viettu-bic requested a review from ntnam-bic August 18, 2025 11:47
Comment thread src/distribute/MiniGamePoolReward.sol Outdated
merkleRoots[_merkleRoot].endTime = _endTime;
emit MerkleRootUpdated(_merkleRoot, _endTime);
} else {
merkleRoots[_merkleRoot] = MerkleRootInfo({

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nên tách quyền update chỉ cho owner với thêm cơ chế để owner có thể paused active root trong emergency case có error trong việc build merkle và đã public. Property exist có thể thay cho paused và check exist nếu merkle root có length bytes

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

access storage trong lib access còn "1CP.access.storage"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

cần căn nhắc đặt trên các selectors do selector sẽ aggregate lại trong diamond

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

T không hiểu lắm đoạn này, nó thiên về config ở script deploy hay gì vậy

…uding addition, update, and pause functionality. Update LibAccess storage key for consistency. Add corresponding error handling and tests for new functionalities.
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