Solved mint function to enable minting from msg.sender

Author: danitome24

packages/foundry/contracts/KDONft.sol

@@ -74,11 +74,9 @@ contract KDONft is ERC721 {
 
   /**
    * @notice Mint an NFT if the user has completed enough tasks to reach a milestone.
-   * @param to Address of the user to mint the NFT for.
    * @param numTasksCompleted Number of tasks the user has completed.
    */
   function mintNft(
-    address to,
     uint256 numTasksCompleted
   ) external hasCompletedMinTasks(numTasksCompleted) {
     if (numTasksCompleted > kiddoPerks.s_childNumTasksCompleted(msg.sender)) {
@@ -87,15 +85,15 @@ contract KDONft is ERC721 {
       );
     }
     TaskMilestone currentTaskMilestone = _getMilestone(numTasksCompleted);
-    if (s_childLastNftMinted[to] == currentTaskMilestone) {
-      revert KDONft__CannotMintNFTMoreThanOnce(to);
+    if (s_childLastNftMinted[msg.sender] == currentTaskMilestone) {
+      revert KDONft__CannotMintNFTMoreThanOnce(msg.sender);
     }
 
-    _safeMint(to, s_nextTokenId);
-    emit KDONftMinted(to, s_nextTokenId);
+    _safeMint(msg.sender, s_nextTokenId);
+    emit KDONftMinted(msg.sender, s_nextTokenId);
     s_nextTokenId++;
 
-    s_childLastNftMinted[to] = currentTaskMilestone;
+    s_childLastNftMinted[msg.sender] = currentTaskMilestone;
   }
 
   /**

packages/foundry/test/KDONft.t.sol

@@ -63,7 +63,8 @@ contract KDONftTest is Test {
   function testMintNftSuccess() public {
     uint256 tasksCompleted = 10;
 
-    kdoNft.mintNft(CHILD_ONE, tasksCompleted);
+    vm.prank(CHILD_ONE);
+    kdoNft.mintNft(tasksCompleted);
 
     assertEq(kdoNft.ownerOf(0), CHILD_ONE);
     assertEq(kdoNft.s_nextTokenId(), 1);
@@ -78,31 +79,36 @@ contract KDONftTest is Test {
     uint256 tasksCompleted = 4;
 
     vm.expectRevert(KDONft.KDONft__MinimumTasksCompletedRequired.selector);
-    kdoNft.mintNft(CHILD_ONE, tasksCompleted);
+    vm.prank(CHILD_ONE);
+    kdoNft.mintNft(tasksCompleted);
   }
 
   function testMintNftRevertIfSameMilestone() public {
     uint256 tasksCompleted = 10;
 
-    kdoNft.mintNft(CHILD_ONE, tasksCompleted);
+    vm.prank(CHILD_ONE);
+    kdoNft.mintNft(tasksCompleted);
 
     vm.expectRevert(
       abi.encodeWithSelector(
         KDONft.KDONft__CannotMintNFTMoreThanOnce.selector, CHILD_ONE
       )
     );
-    kdoNft.mintNft(CHILD_ONE, tasksCompleted);
+    vm.prank(CHILD_ONE);
+    kdoNft.mintNft(tasksCompleted);
   }
 
   function testMintNftMultipleMilestones() public {
-    kdoNft.mintNft(CHILD_ONE, 10);
+    vm.prank(CHILD_ONE);
+    kdoNft.mintNft(10);
     assertEq(kdoNft.ownerOf(0), CHILD_ONE);
     assertEq(
       uint256(kdoNft.s_childLastNftMinted(CHILD_ONE)),
       uint256(KDONft.TaskMilestone.TEN)
     );
 
-    kdoNft.mintNft(CHILD_ONE, 20);
+    vm.prank(CHILD_ONE);
+    kdoNft.mintNft(20);
     assertEq(kdoNft.ownerOf(1), CHILD_ONE);
     assertEq(
       uint256(kdoNft.s_childLastNftMinted(CHILD_ONE)),

packages/nextjs/components/kiddo-perks/ParentDashboard/ChildSummaryCard.tsx

@@ -21,7 +21,7 @@ export const ChildSummaryCard = ({ child, totalTasks }: ChildSummaryCardProps) =
         </h2>
         <p className="text-lg text-right m-0">{formattedTokenBalance} KDO</p>
         <p className="m-0">
-          Tasks completed: {taskByChild.completedTasksNumber}/{totalTasks?.toString()}
+          Tasks completed: {taskByChild.uniqueCompletedTasksNumber}/{totalTasks?.toString()}
         </p>
 
         <div className="card-actions justify-end">

packages/nextjs/contracts/deployedContracts.ts

@@ -7,7 +7,7 @@ import { GenericContractsDeclaration } from "~~/utils/scaffold-eth/contract";
 const deployedContracts = {
   31337: {
     KDOToken: {
-      address: "0xe1da8919f262ee86f9be05059c9280142cf23f48",
+      address: "0x82dc47734901ee7d4f4232f398752cb9dd5daccc",
       abi: [
         {
           type: "constructor",
@@ -450,7 +450,7 @@ const deployedContracts = {
       },
     },
     KiddoPerks: {
-      address: "0x0c8e79f3534b00d9a3d4a856b665bf4ebc22f2ba",
+      address: "0x196dbcbb54b8ec4958c959d8949ebfe87ac2aaaf",
       abi: [
         {
           type: "constructor",
@@ -1451,7 +1451,7 @@ const deployedContracts = {
       },
     },
     KDONft: {
-      address: "0xed1db453c3156ff3155a97ad217b3087d5dc5f6e",
+      address: "0x82c6d3ed4cd33d8ec1e51d0b5cc1d822eaa0c3dc",
       abi: [
         {
           type: "constructor",
@@ -1592,11 +1592,6 @@ const deployedContracts = {
           type: "function",
           name: "mintNft",
           inputs: [
-            {
-              name: "to",
-              type: "address",
-              internalType: "address",
-            },
             {
               name: "numTasksCompleted",
               type: "uint256",