Skip to content

feat: set base token URI#27

Merged
huyhuynh3103 merged 5 commits intomainfrom
feat/set-base-token-uri
Jul 2, 2025
Merged

feat: set base token URI#27
huyhuynh3103 merged 5 commits intomainfrom
feat/set-base-token-uri

Conversation

@huyhuynh3103
Copy link
Copy Markdown
Contributor

Description

Checklist

  • I have clearly commented on all the main functions following the NatSpec Format
  • The box that allows repo maintainers to update this PR is checked
  • I tested locally to make sure this feature/fix works

Comment thread src/upgradeable/ERC721PresetMinterPauserAutoIdCustomizedUpgradeable.sol Outdated
) internal {
require(bytes(baseTokenURI).length > 0, InvalidBaseTokenURI());

_baseTokenURI = baseTokenURI;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

should fire event?

@huyhuynh3103 huyhuynh3103 merged commit b4e3665 into main Jul 2, 2025
@huyhuynh3103 huyhuynh3103 deleted the feat/set-base-token-uri branch July 2, 2025 05:26
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.

2 participants