Skip to content

feat: add data to mint and mintBatch#199

Merged
maxnorm merged 1 commit into
Perfect-Abstractions:mainfrom
akronim26:feat/erc-1155
Nov 14, 2025
Merged

feat: add data to mint and mintBatch#199
maxnorm merged 1 commit into
Perfect-Abstractions:mainfrom
akronim26:feat/erc-1155

Conversation

@akronim26

@akronim26 akronim26 commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

Summary

Add data param in mint and mintBatch function in LibERC1155.sol so that if the receiver is a contract, it can run custom logic.

Changes Made

Add data param in mint and mintBatch function in LibERC1155.sol

Checklist

Before submitting this PR, please ensure:

  • Code follows the Solidity feature ban - No inheritance, constructors, modifiers, public/private variables, external library functions, using for directives, or selfdestruct

  • Code follows Design Principles - Readable, uses diamond storage, favors composition over inheritance

  • Code matches the codebase style - Consistent formatting, documentation, and patterns (e.g. ERC20Facet.sol)

  • Code is formatted with forge fmt

  • Tests are included - All new functionality has comprehensive tests

  • All tests pass - Run forge test and ensure everything works

  • Documentation updated - If applicable, update relevant documentation

Make sure to follow the CONTRIBUTING.md guidelines.

Additional Notes

@netlify

netlify Bot commented Nov 13, 2025

Copy link
Copy Markdown

‼️ Deploy request for compose-diamonds rejected.

Name Link
🔨 Latest commit 9bfc237

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report

Coverage

Metric Coverage Details
Lines 61% 934/1540 lines
Functions 68% 183/268 functions
Branches 42% 108/260 branches

Last updated: Thu, 13 Nov 2025 18:04:37 GMT for commit 9bfc237

@github-actions

Copy link
Copy Markdown
Contributor

Gas Report

No gas usage changes detected between main and feat/erc-1155.

All functions maintain the same gas costs. ✅

Last updated: Thu, 13 Nov 2025 18:05:06 GMT for commit 9bfc237

@maxnorm

maxnorm commented Nov 14, 2025

Copy link
Copy Markdown
Collaborator

Thanks @akronim26!

@maxnorm maxnorm merged commit 4bd6d03 into Perfect-Abstractions:main Nov 14, 2025
4 checks passed
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.

[Feature]: ERC1155 - Add data param to mint

2 participants