Add withdrawer role to allow withdraw

[daveroga]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 20436939753

Details

• 7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.2%) to 85.977%

---

Totals
Change from base Build 20378025506:	0.2%
Covered Lines:	1412
Relevant Lines:	1543

---

💛 - Coveralls

[Copilot]

Pull request overview

This PR adds role-based access control to the MCPayment contract by introducing a WITHDRAWER_ROLE that allows designated accounts to withdraw owner funds in addition to the owner themselves. The implementation integrates OpenZeppelin's AccessControlUpgradeable and modifies withdrawal functions to support this new role.

Key changes:

• Adds AccessControlUpgradeable inheritance and WITHDRAWER_ROLE constant
• Implements setAdminRole function and onlyWithdrawerRoleOrOwner modifier for role-based withdrawals
• Modifies ownerWithdraw and ownerERC20Withdraw functions to allow WITHDRAWER_ROLE accounts to withdraw funds to their own addresses

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 12 comments.

File	Description
contracts/payment/MCPayment.sol	Integrates AccessControlUpgradeable, adds WITHDRAWER_ROLE with associated modifier, implements setAdminRole function, and updates withdrawal functions to support the new role
test/payment/mc-payment.test.ts	Adds comprehensive test case for WITHDRAWER_ROLE functionality and updates existing test to reflect new AccessControl error types

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.