Fixed pop confirm's ok button on product and domain page, fixed product updating issue, remvoed console.log that was added for debugging prviously

Author: ydahal1

Diff for: Tombolo/client-reactjs/src/components/admin/Integrations/asr/DomainsTab.jsx

@@ -42,7 +42,7 @@ const DomainsTab = ({ domains, setSelectedDomain, setDomainModalOpen }) => {
             title="Are you sure you want to delete this domain?"
             onConfirm={() => handleDelete(record)}
             okText="Yes"
-            okButtonProps={{ type: 'danger' }}>
+            okButtonProps={{ type: 'primary', danger: true }}>
             <DeleteOutlined style={{ color: 'var(--primary)' }} type="delete" />
           </Popconfirm>
         </Space>

Diff for: Tombolo/client-reactjs/src/components/admin/Integrations/asr/ProductModal.jsx

@@ -93,6 +93,7 @@ const ProductModal = ({
     try {
       const payload = {
         ...values,
+        id: selectedProduct.id,
         updatedBy: { lastName: user.lastName, firstName: user.firstName, email: user.email },
       };
       await updateProduct({ id: selectedProduct.id, payload });
@@ -112,7 +113,6 @@ const ProductModal = ({
   // JSX
   return (
     <Modal
-      title="Add product"
       open={productModalOpen}
       okText={selectedProduct ? 'Update' : 'Save'}
       onOk={handleOk}

Diff for: Tombolo/client-reactjs/src/components/admin/Integrations/asr/ProductsTab.jsx

@@ -98,7 +98,7 @@ const ProductsTab = ({ products, setSelectedProduct, setProductModalOpen, setPro
           <Popconfirm
             title="Are you sure you want to delete this product?"
             onConfirm={() => handleDelete(record)}
-            okButtonProps={{ type: 'danger' }}>
+            okButtonProps={{ type: 'primary', danger: true }}>
             <DeleteOutlined type="delete" />
           </Popconfirm>
         </Space>

Diff for: Tombolo/server/routes/asr/read.js

@@ -254,6 +254,7 @@ router.get("/products/", async(req, res) => {
              attributes: ["id", "name"],
            },
          ],
+         order: [["createdAt", "DESC"]],
          raw: true,
        });
 
@@ -283,7 +284,7 @@ router.get("/productsOnly/", async(req, res) => {
 router.put(
   "/products/:id",
   [
-    param("id").notEmpty().isUUID().withMessage("ID must be a UUID"),
+    param("id").notEmpty().isInt().withMessage("ID must be an integer"),
     body("name").notEmpty().isString().withMessage("Product name is required"),
     body("shortCode").notEmpty().isString().withMessage("Short code is required"),
     body("tier").notEmpty().isInt().withMessage("Tier is required"),

Diff for: Tombolo/server/routes/monitorings/read.js

@@ -13,9 +13,6 @@ const MonitoringTypes = models.monitoring_types;
 // Route to get all monitoring types
 router.get("/", async(req, res) => {
     try{
-        console.log('------------------------------------------');
-        console.dir("GET ALL MONITORINGS")
-        console.log('------------------------------------------');
         const monitoringTypes = await MonitoringTypes.findAll();
         res.status(200).json(monitoringTypes);
     }catch(err){