Skip to content

v2 · Feature 1 — Reduce Max Template Supply - #1

Closed
robrigo wants to merge 3 commits into
mainfrom
feat/v2-1-reduce-max-supply
Closed

v2 · Feature 1 — Reduce Max Template Supply#1
robrigo wants to merge 3 commits into
mainfrom
feat/v2-1-reduce-max-supply

Conversation

@robrigo

@robrigo robrigo commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Reduces maximum template supply.


Mirrored into the canonical atomicassets org for the AtomicAssets v2 release + audit.
Original: wax-office-of-inspector-general/atomicassets-contract#3

⚠️ Hold — do not merge to master yet. Kept on a feature branch to avoid prod integration risk; pending the comprehensive v2 audit.

@robrigo
robrigo marked this pull request as ready for review June 2, 2026 20:49
@robrigo
robrigo requested a review from Copilot June 2, 2026 20:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a new contract action to reduce a template’s max_supply, allowing authorized collection editors to lower the remaining mintable supply for an existing template as part of the AtomicAssets v2 feature set.

Changes:

  • Added redtemplmax action to reduce a template’s max_supply while ensuring it cannot go below issued_supply.
  • Exposed the new action in the contract header so it becomes part of the contract’s public API/ABI.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/atomicassets.cpp Implements redtemplmax action logic and template table mutation.
include/atomicassets.hpp Declares the new redtemplmax action in the contract interface.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/atomicassets.cpp
Comment on lines +518 to +521
require_auth(authorized_editor);

auto collection_itr = collections.require_find(collection_name.value,
"No collection with this name exists");
@robrigo

robrigo commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #9, which is merged. This feature is on main: redtemplmax is declared in include/atomicassets.hpp and shipped in v2.0.0-rc4 and rc5. The integration branch carried the feature under its own commits rather than these, so this PR could not close automatically. Branch kept for history.

@robrigo robrigo closed this Jul 16, 2026
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