diff --git a/app/Controllers/Products.php b/app/Controllers/Products.php
index 9ade4f0..4918b0d 100644
--- a/app/Controllers/Products.php
+++ b/app/Controllers/Products.php
@@ -65,7 +65,7 @@ class Products extends BaseController
         $ProductModel = new ProductModel();
     
         $tax= $this->request->getPost('tax')/2;
-        
+        // print_r($this->request->getPost());die;
         $data = [
             'product_name' => $this->request->getPost('productname'),
             'product_category' => $this->request->getPost('productcategory'),
@@ -82,6 +82,8 @@ class Products extends BaseController
             'vendor' => $this->request->getPost('vendor'),
             'purchase_order_level' => $this->request->getPost('purchase_order_level'),
             'quality' => $this->request->getPost('quality'), 
+            'make_id' => $this->request->getPost('make'), 
+            'models_id' => json_encode($this->request->getPost('model')), 
             'isactive' => 1 ,
             'branch_id' => $this->session->get('logged_user_branch_id')
         ];
@@ -124,26 +126,17 @@ class Products extends BaseController
         // Retrieve the make_id from the AJAX request
         public function get_vendors_by_model()
         {
-            $make_id = $this->request->getPost('make_id');
+            
             $model_ids = $this->request->getPost('model_ids');
-            // print_r( $model_ids);die;
-    
-            // If make_id or model_ids is not provided, return an empty response
            
-    
-            // Create an instance of VendorModel
             $vendorModel = new VendorModel();
-    
-            // Call the get_vendors_by_model method from the model
-            $vendors = $vendorModel->get_vendors_by_model($make_id, $model_ids);
-    // print_r($vendors);
-            // Return the response as JSON
-            // return $this->response->setJSON(['success' => true, 'message' => 'Client saved successfully.']);
+            $vendors = $vendorModel->get_vendors_by_model( $model_ids);
+
             if ($vendors) {
-                // Return the retrieved vendors as JSON response
+                
                 return $this->response->setJSON($vendors);
             } else {
-                // If no vendors were found, return an empty response or handle the error accordingly
+               
                 return $this->response->setJSON([]);
             }
         }
diff --git a/app/Models/ProductModel.php b/app/Models/ProductModel.php
index a0f7a1f..63c54ff 100644
--- a/app/Models/ProductModel.php
+++ b/app/Models/ProductModel.php
@@ -5,7 +5,7 @@ class ProductModel extends Model
 {
     protected $table = 'products';
     protected $primaryKey = 'product_id';
-    protected $allowedFields = ['product_id','quality','product_name','branch_id','search_tag','product_category','mfr_part_no','purchase_order_level','serial_no','unit_price','commision_rate','sgst','cgst','purchase_cost','usage_unit','vendor','qty_stock','reorder_level','handler','qty_in_demand','product_image','description','isactive'];
+    protected $allowedFields = ['product_id','make_id','models_id','quality','product_name','branch_id','search_tag','product_category','mfr_part_no','purchase_order_level','serial_no','unit_price','commision_rate','sgst','cgst','purchase_cost','usage_unit','vendor','qty_stock','reorder_level','handler','qty_in_demand','product_image','description','isactive'];
     
     
     public function getTax()
diff --git a/app/Views/client_form.php b/app/Views/client_form.php
index a8c072a..e962a9a 100644
--- a/app/Views/client_form.php
+++ b/app/Views/client_form.php
@@ -21,6 +21,7 @@
             <div class="card-body">
                 
                 <form action="<?= base_url() . "add_client"; ?>" method="post">
+                <h4 class="header-title">Client Details :</h4>
                     <div class="form-row">
                         <div class="form-group col-md-4">
                             <label for="inputEmail4" class="col-form-label">Client Name<span class="text-danger">*</span></label>
@@ -30,6 +31,16 @@
                             <label for="inputAddress" class="col-form-label">Mobile<span class="text-danger">*</span></label>
                             <input type="text" class="form-control" name="mobile" placeholder="Mobile" value="<?= isset($client['mobile_no']) ? $client['mobile_no'] : '' ?>" maxlength="10" minlength="10" onkeypress = "return onlyNumbers(event)" required/>
                         </div>
+                        <div class="form-group col-md-4">
+                        <label for="inputPassword4" class="col-form-label">Client Type<span class="text-danger">*</span></label>
+                        <select class="form-control status-select" name="clienttype" required data-toggle="select2" >
+                        <!-- <option value="">Select a Type</option> -->
+                        <option value="Direct" <?= isset($client['client_type']) && $client['client_type'] === 'Direct' ? 'selected' : '' ?>>Direct</option>
+                        <option value="Mechanic" <?= isset($client['client_type']) && $client['client_type'] === 'Mechanic' ? 'selected' : '' ?>>Mechanic</option>
+                        <option value="Exp" <?= isset($client['client_type']) && $client['client_type'] === 'Exp' ? 'selected' : '' ?>>Exp</option>
+                
+                        </select>
+                        </div>
                         <div class="form-group col-md-4">
                             <label for="inputEmail4" class="col-form-label">Email</label>
                             <input type="email" class="form-control" name="email" placeholder="Email" value="<?= isset($client['email']) ? $client['email'] : '' ?>" >
@@ -38,23 +49,10 @@
                             <label for="inputAddress" class="col-form-label">GST Number<span class="text-danger"></span></label>
                             <input type="text" class="form-control" name="gstnumber" placeholder="GST Number" value="<?= isset($client['gstnumber']) ? $client['gstnumber'] : '' ?>"pattern="[0-9]{2}[A-Z]{5}[0-9]{4}[A-Z]{1}[0-9]{1}[Z]{1}[0-9A-Z]{1}" title="Enter a valid GST Number" />
                         </div>
-                        <div class="form-group col-md-6">
-                <label for="inputPassword4" class="col-form-label">Client Type<span class="text-danger">*</span></label>
-                <select class="form-control status-select" name="clienttype" required data-toggle="select2" >
-                 <!-- <option value="">Select a Status</option> -->
-                <option value="Direct" <?= isset($client['client_type']) && $client['client_type'] === 'Direct' ? 'selected' : '' ?>>Direct</option>
-                 <option value="Mechanic" <?= isset($client['client_type']) && $client['client_type'] === 'Mechanic' ? 'selected' : '' ?>>Mechanic</option>
-                 <option value="Exp" <?= isset($client['client_type']) && $client['client_type'] === 'Exp' ? 'selected' : '' ?>>Exp</option>
-        
-                 </select>
-                </div>
-
-                    
-                       
-                       
+                        
                     </div>
                     <input type="hidden" id="client_id" name="client_id" value="<?= isset($client['client_id']) ? $client['client_id'] : '' ?>">
-                    <h4 class="header-title">Address Information :</h4>
+                    </br><h4 class="header-title">Address Information :</h4>
                     <div class="form-row">
                            <div class="form-group col-md-6">
                             <label for="inputPassword4" class="col-form-label">Address</label>
diff --git a/app/Views/product_form.php b/app/Views/product_form.php
index e79bb74..237b6aa 100644
--- a/app/Views/product_form.php
+++ b/app/Views/product_form.php
@@ -26,6 +26,7 @@
                     <div class="form-group col-md-4">
                             <label for="make" class="col-form-label">Make<span class="text-danger">*</span></label>
                             <select class="form-control SelExample" id="make" name="make" required>
+                                 <option value="">Select a Make</option>
                                 <!-- Options for make -->
                                 <?php foreach ($make as $value): ?>
                                     <option value="<?= $value['make_id'] ?>" <?= isset($vendor['make_ids']) && in_array($value['make_id'], $vendor['make_ids']) ? 'selected' : '' ?>><?= $value['make'] ?></option>
@@ -43,23 +44,23 @@
                             <input type="text" class="form-control" name="productname" placeholder="Product Name" value="<?= isset($products['product_name']) ? $products['product_name'] : '' ?>" required>
                         </div>
                         <div class="form-group col-md-4">
-    <label for="vendor" class="col-form-label">Vendor<span class="text-danger">*</span></label>
-    <select class="form-control" id="vendor" name="vendor" required>
-        <!-- Placeholder option -->
-        <option value="">Select Vendor</option>
-    </select>
-</div>
-<div class="form-group col-md-4">
-    <label for="vendor" class="col-form-label">Quality<span class="text-danger">*</span></label>
-    <select class="form-control  " name="quality" required data-toggle="select2" >
-        <!-- <option value="">Select a Status</option> -->
-        <option value="Q1" <?= isset($products['quality']) && $products['quality'] === 'Q1' ? 'selected' : '' ?>>Q1</option>
-        <option value="Q2" <?= isset($products['quality']) && $products['quality'] === 'Q2' ? 'selected' : '' ?>>Q2 </option>
-        <option value="Q3" <?= isset($products['quality']) && $products['quality'] === 'Q3' ? 'selected' : '' ?>>Q3</option>
-        <option value="Q4" <?= isset($products['quality']) && $products['quality'] === 'Q4' ? 'selected' : '' ?>>Q4</option>
-    </select>
-</div>
-                        
+                            <label for="vendor" class="col-form-label">Vendor<span class="text-danger">*</span></label>
+                            <select class="form-control" id="vendor" name="vendor" required>
+                                <!-- Placeholder option -->
+                                <option value="">Select Vendor</option>
+                            </select>
+                        </div>
+                        <div class="form-group col-md-4">
+                            <label for="vendor" class="col-form-label">Quality<span class="text-danger">*</span></label>
+                            <select class="form-control  " name="quality" required data-toggle="select2" >
+                                <!-- <option value="">Select a Status</option> -->
+                                <option value="Q1" <?= isset($products['quality']) && $products['quality'] === 'Q1' ? 'selected' : '' ?>>Q1</option>
+                                <option value="Q2" <?= isset($products['quality']) && $products['quality'] === 'Q2' ? 'selected' : '' ?>>Q2 </option>
+                                <option value="Q3" <?= isset($products['quality']) && $products['quality'] === 'Q3' ? 'selected' : '' ?>>Q3</option>
+                                <option value="Q4" <?= isset($products['quality']) && $products['quality'] === 'Q4' ? 'selected' : '' ?>>Q4</option>
+                            </select>
+                        </div>
+                                                
                         <div class="form-group col-md-4">
                             <label for="inputAddress" class="col-form-label">Manfacturer Part Number</label>
                             <input type="text" class="form-control" name="mfr_part_no" placeholder="Manfacturer Part Number" value="<?= isset($products['mfr_part_no']) ? $products['mfr_part_no'] : '' ?>" >
@@ -200,6 +201,7 @@ $(document).ready(function() {
                     // Clear existing options
                     $('#model').empty();
                     // Append new options
+                    $("#model").append('<option value=""> Select a Model </option>');
                     $.each(response, function(index, model) {
                         $('#model').append('<option value="' + model.model_id + '">' + model.model_name + '</option>');
                     });
@@ -218,7 +220,7 @@ $(document).ready(function() {
 
 $(document).ready(function() {
     $('#model').change(function() {
-        var make_id = $(this).val(); // Get the selected make ID
+
         var model_ids = $('#model').val(); // Get the selected model IDs
 console.log(model_ids);
 
@@ -227,14 +229,14 @@ console.log(model_ids);
             url: '<?php echo base_url('get_vendors_by_model') ?>', // URL to your CodeIgniter controller method
             type: 'POST',
             dataType: 'json',
-            data: { make_id: make_id, model_ids: model_ids }, // Send the selected make ID and model IDs to the controller
+            data: {  model_ids: model_ids }, // Send the selected make ID and model IDs to the controller
             success: function(response) {
                 // Update vendor dropdown based on the response
                 if (response) {
                     // Clear existing options
                     $('#vendor').empty();
                     // Append new options
-                    $('#vendor').append('<option value>"hello"</option>');
+                    $('#vendor').append('<option value>Select Vendor</option>');
                     $.each(response, function(index, vendor) {
                         $('#vendor').append('<option value="' + vendor.vendor_id + '">' + vendor.vendor_name + '</option>');
                     });
diff --git a/app/Views/vehicle_form.php b/app/Views/vehicle_form.php
index 41b984d..588e1a2 100644
--- a/app/Views/vehicle_form.php
+++ b/app/Views/vehicle_form.php
@@ -20,10 +20,10 @@
                 
                 <form action="<?= base_url() . "add_vehicle"; ?>" method="post">
                
-                    <h4 class="header-title">Vehicle Details :</h4><br>
+                    <h4 class="header-title">Vehicle Details :</h4>
                     <div class="form-row">
                     <div class="form-group col-md-4">
-                            <label for="inputPassword4" class="col-form-label">Make</label>
+                            <label for="inputPassword4" class="col-form-label">Make<span class="text-danger">*</span></label>
                             <select class="form-control SelExample" name="make" id="make"   <?= isset($vehicle['make']) ? '' : 'required' ?>>
                             <?= isset($vehicle['make']) ? '' : '<option value="">Select a Make</option>' ?>
                             <?php foreach ($makeData as $value) : ?>
@@ -36,7 +36,7 @@
                             </select>
                         </div>
                         <div class="form-group col-md-4">
-                            <label for="inputPassword4" class="col-form-label">Model</label>
+                            <label for="inputPassword4" class="col-form-label">Model<span class="text-danger">*</span></label>
                             <select class="form-control SelExample" name="model" id="model"   <?= isset($vehicle['model']) ? '' : 'required' ?>>
                             <?php if (isset($vehicle["model_name"])) : ?>
                                 <option value="<?= $vehicle["model"] ?>" >
@@ -46,7 +46,7 @@
                             </select>
                         </div>
                         <div class="form-group col-md-4">
-                            <label for="inputPassword4" class="col-form-label">Reg Number</label>
+                            <label for="inputPassword4" class="col-form-label">Reg Number<span class="text-danger">*</span></label>
                             <input type="text" class="form-control" name="regno" id="regno" placeholder="Reg Number"value="<?= isset($vehicle['reg_no']) ? $vehicle['reg_no'] : '' ?>" required>
                         </div>
                         <div class="form-group col-md-4">
@@ -58,10 +58,11 @@
                             <input type="text" class="form-control" name="colour" placeholder="Colour"value="<?= isset($vehicle['colour']) ? $vehicle['colour'] : '' ?>" >
                         </div>
                     </div>
-                    <h4 class="header-title">Client Details :</h4><br>
+                    </br>
+                    <h4 class="header-title">Client Details :</h4>
                     <div class="form-row">
                         <div class="form-group col-md-4">
-                        <label for="inputEmail4" class="col-form-label">Client Name</label>
+                        <label for="inputEmail4" class="col-form-label">Client Name<span class="text-danger">*</span></label>
                         <?= isset($vehicle['client_id']) ? '' : '<i type="button" class="fe-plus-circle" id="addClientModalButton" style="font-size: 18px;" data-toggle="modal" data-target="#addClientModal" title="Add Client"></i>' ?>
                         <select class="form-control cleint-select SelExample" id="clientData" name="client_name" required data-toggle="select2" >
                         <option value="">Select a Client</option>
@@ -74,14 +75,15 @@
                         <?php endforeach; ?>
                         </select>
                         </div>
+                        <div class="form-group col-md-4">
+                            <label for="inputAddress" class="col-form-label">Mobile<span class="text-danger">*</span></label>
+                            <input type="text" class="form-control" name="mobile_no" placeholder="Mobile" value="<?= isset($vehicle['mobile_no']) ? $vehicle['mobile_no'] : '' ?>" required maxlength="10" minlength="10">
+                        </div>
                         <div class="form-group col-md-4">
                             <label for="inputEmail4" class="col-form-label">Email</label>
                             <input type="email" class="form-control" name="email" placeholder="Email" value="<?= isset($vehicle['email']) ? $vehicle['email'] : '' ?>" >
                         </div>
-                        <div class="form-group col-md-4">
-                            <label for="inputAddress" class="col-form-label">Mobile</label>
-                            <input type="text" class="form-control" name="mobile_no" placeholder="Mobile" value="<?= isset($vehicle['mobile_no']) ? $vehicle['mobile_no'] : '' ?>" required maxlength="10" minlength="10">
-                        </div>
+                       
                         <div class="form-group col-md-3">
                             <label for="inputPassword4" class="col-form-label">Address</label>
                             <input type="text" class="form-control" name="address" placeholder="Address"value="<?= isset($vehicle['address']) ? $vehicle['address'] : '' ?>" >