Skip to content

Conversation

@Pronoss
Copy link
Contributor

@Pronoss Pronoss commented Apr 22, 2025

What changed? Why?

  • Added Ownable import from OpenZeppelin and extended the MyERC721Token contract with Ownable.
  • Updated the switchURI() function to be onlyOwner to enforce access control.
  • This change ensures only the contract owner can modify the active NFT metadata base URI (BAYC/DOODLES), resolving the security TODO.

@vercel
Copy link

vercel bot commented Apr 22, 2025

@Pronoss is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Apr 22, 2025

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@wbnns wbnns added the base docs review To be reviewed by the Base docs team label May 8, 2025
@Pronoss
Copy link
Contributor Author

Pronoss commented Aug 5, 2025

Hi @wbnns – just a friendly reminder about PR #2240. It updates the ERC-721 example to import Ownable and restricts switchURI() to only the owner. Whenever you get a moment to take a look, I’d appreciate it. Thanks! 🙏

@Pronoss Pronoss closed this Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

base docs review To be reviewed by the Base docs team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants