Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unable to use latest access-based ERC721ACommon with ArbitraryPriceSeller #90

Open
wattsyart opened this issue Jun 26, 2023 · 0 comments
Open

Unable to use latest access-based ERC721ACommon with ArbitraryPriceSeller #90

wattsyart opened this issue Jun 26, 2023 · 0 comments

Comments

@wattsyart
Copy link

image

It seems like the latest iteration can't blend ERC721ACommon with ArbitraryPriceSeller because they both use different flavours of Pausable that don't have virtual functions. Each one wants to implement them with different restriction sets.

Normally you'd just override them both in the consuming contract and choose. However, they are not marked virtual, so this is not possible.

Are you able to make them virtual so consumers can override them?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wattsyart