Skip to content

Sync slice/internal → main - #14

Open
github-actions[bot] wants to merge 42 commits into
mainfrom
slice/internal
Open

Sync slice/internal → main#14
github-actions[bot] wants to merge 42 commits into
mainfrom
slice/internal

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown

Automated PR – triggered by commit 98eb7fe


PR-Codex overview

This PR focuses on refactoring and enhancing the pricing and action hooks in the smart contract codebase, improving the structure and functionality of various product and pricing strategies.

Detailed summary

  • Removed obsolete files related to LinearVRGDAParams and LogisticVRGDAParams.
  • Updated import statements for better clarity and organization.
  • Modified function signatures in several contracts to include variantId.
  • Enhanced isPurchaseAllowed functions to handle variants.
  • Refactored configureProduct functions to accommodate variant-specific parameters.
  • Introduced new mappings for handling variant-specific configurations in actions and pricing.
  • Improved documentation and comments for clarity on new structures and functions.
  • Updated testing files to reflect changes in function signatures and behaviors.

The following files were skipped due to too many changes: src/hooks/actions/NFTGated/NFTGated.sol, src/hooks/pricing/VRGDA/LogisticVRGDAPrices/LogisticVRGDAPrices.sol, src/hooks/actions/ERC721Mint/ERC721Mint.sol, script/ScriptUtils.sol, test/pricing/VRGDA/LogisticVRGDA.t.sol, test/pricing/VRGDA/LinearVRGDA.t.sol, src/hooks/actions/ERC20Mint/ERC20Mint.sol, deployments/addresses.json, script/generate-hook.sh, test/actions/ERC20Gated/ERC20Gated.t.sol, test/actions/NFTGated/NFTGated.t.sol, test/actions/ERC20Mint/ERC20Mint.t.sol, test/pricing/TieredDiscount/NFTDiscount.t.sol, test/actions/ERC721Mint/ERC721Mint.t.sol, test/pricingActions/FirstForFree/FirstForFree.t.sol

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

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