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.

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

Update dependency registry with supported contract overrides #1730

Conversation

ryley-o
Copy link
Contributor

@ryley-o ryley-o commented Nov 26, 2024

Description of the change

Update dependency registry with supported contract overrides.

fixes https://linear.app/art-blocks/issue/PLT-901/update-dependency-registry-with-supported-contract-overrides

This allows the dependency registry to add supported contracts in addition to what is defined on the core registry.

This supports the use case where a core contract is banned from minting on the shared minter suite, but has projects should be supported by the dependency registry (e.g. safe fallback if compromised partner admin keys).

Note: an event is also added when core registry is set, for future indexing capabilities.

TODO

@ryley-o ryley-o added the do not merge Not auto-merge-able label Nov 26, 2024
@ryley-o ryley-o marked this pull request as ready for review November 26, 2024 00:44
@ryley-o ryley-o requested a review from a team as a code owner November 26, 2024 00:45
@ryley-o ryley-o changed the base branch from main to ryley/plt-890-follow-on-inject-dependencies-for-flex November 26, 2024 05:05
Copy link
Contributor

@lindsgil lindsgil left a comment

Choose a reason for hiding this comment

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

In the future do you think it makes sense to explore role based access control for contract addresses? 🤔

@ryley-o ryley-o mentioned this pull request Nov 26, 2024
1 task
@ryley-o
Copy link
Contributor Author

ryley-o commented Nov 26, 2024

In the future do you think it makes sense to explore role based access control for contract addresses? 🤔

I think there is definitely a healthy amount of improvement we can make around access controls and permission granularity (e.g. at project level) in the future, but for now this is a good start 😃

Base automatically changed from ryley/plt-890-follow-on-inject-dependencies-for-flex to main November 26, 2024 16:22
@ryley-o ryley-o removed the do not merge Not auto-merge-able label Nov 26, 2024
@ryley-o ryley-o merged commit be05ea9 into main Nov 26, 2024
13 of 14 checks passed
@ryley-o ryley-o deleted the ryley/plt-901-update-dependency-registry-with-supported-contract-overrides branch November 26, 2024 17:38
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.

3 participants