feat: add multiple minters

Author: Yash Agrawal

contracts/SBTToken.sol

@@ -14,6 +14,8 @@ contract SBTToken is ISBTToken, ERC721EnumerableUpgradeable {
 
 	/// @dev Account with proxy adming rights.
 	address private _proxyAdmin;
+	/// @dev EOA with rights to allow(add)/disallow(remove) minter.
+	address private _minterUpdater;
 
 	/// @dev EOA with minting rights.
 	mapping(address => bool) private _minters;
@@ -25,6 +27,11 @@ contract SBTToken is ISBTToken, ERC721EnumerableUpgradeable {
 		_;
 	}
 
+	modifier onlyMinterUpdater() {
+		require(_msgSender() == _minterUpdater, "Not minter updater");
+		_;
+	}
+
 	function _setTokenURI(uint256 tokenId, bytes memory metadata) private {
 		_sbtdata[tokenId] = metadata;
 		emit SetSBTTokenURI(tokenId, metadata);
@@ -51,9 +58,13 @@ contract SBTToken is ISBTToken, ERC721EnumerableUpgradeable {
 		}
 	}
 
-	function initialize(address[] memory minters) external initializer {
+	function initialize(
+		address minterUpdater,
+		address[] memory minters
+	) external initializer {
 		__ERC721_init("Dev Protocol SBT V1", "DEV-SBT-V1");
 
+		_minterUpdater = minterUpdater;
 		for (uint256 i = 0; i < minters.length; i++) {
 			_minters[minters[i]] = true;
 		}
@@ -65,6 +76,16 @@ contract SBTToken is ISBTToken, ERC721EnumerableUpgradeable {
 		emit SetProxyAdmin(proxyAdmin);
 	}
 
+	function addMinter(address minter) external override onlyMinterUpdater {
+		_minters[minter] = true;
+		emit MinterAdded(minter);
+	}
+
+	function removeMinter(address minter) external override onlyMinterUpdater {
+		_minters[minter] = false;
+		emit MinterRemoved(minter);
+	}
+
 	function setTokenURI(
 		uint256 tokenId,
 		bytes memory metadata

contracts/interfaces/ISBTToken.sol

@@ -35,6 +35,18 @@ interface ISBTToken {
 	 */
 	event SetSBTTokenURI(uint256 tokenId, bytes data);
 
+	/*
+	 * @dev The event fired when a EOA is given minter right.
+	 * @param minter The address given the minter right.
+	 */
+	event MinterAdded(address minter);
+
+	/*
+	 * @dev The event fired when a EOA is revoked from minter right.
+	 * @param minter The address revoked the minter right.
+	 */
+	event MinterRemoved(address minter);
+
 	/*
 	 * @dev Creates the new staking position for the caller.
 	 * Mint must be called by the minter address.
@@ -46,6 +58,20 @@ interface ISBTToken {
 		bytes memory metadata
 	) external returns (uint256);
 
+	/*
+	 * @dev Gives minter rights to EOA.
+	 * @notice must be called by the minterUpdater address.
+	 * @param minter The address who is to be given the minting rights.
+	 */
+	function addMinter(address minter) external;
+
+	/*
+	 * @dev Revokes minter rights to EOA.
+	 * @notice must be called by the minterUpdater address.
+	 * @param minter The address who is to be revoked from the minting rights.
+	 */
+	function removeMinter(address minter) external;
+
 	/*
 	 * @dev Sets the token URI image for a token.
 	 * @notice must be called by the minter address.