You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description
The CCIPReceive abstract contract is incompatible with the AccessControlDefaultAdminRules from OpenZeppelin, when implementing both, we need to override the supportsInterface(bytes4) function, but they have different state mutability, CCIPReceive uses pure, and AccessControlDefaultAdminRules uses view. So we can't override it and use super.supportsInterface(bytes4) for example
- make getRouter virtual to allow overwrites
- Makes supportsInterface view, to resolve override issues with other
view-supportsInterface functions
Closes#848
Description
The CCIPReceive abstract contract is incompatible with the AccessControlDefaultAdminRules from OpenZeppelin, when implementing both, we need to override the
supportsInterface(bytes4)
function, but they have different state mutability, CCIPReceive uses pure, and AccessControlDefaultAdminRules uses view. So we can't override it and usesuper.supportsInterface(bytes4)
for exampleSteps to Reproduce
Additional Information
Perhaps change the state mutability to view? because pure can override view, but the inverse is not possible
The text was updated successfully, but these errors were encountered: