Merge pull request 'merge' (#1) from develop into version-14

Reviewed-on: #1

manufacturing_overview/hooks.py

@@ -15,13 +15,7 @@ required_apps = ["erpnext"]
 
 # include js, css files in header of desk.html
 # app_include_css = "/assets/manufacturing_overview/css/manufacturing_overview.css"
-app_include_js = [
-    "drawer_bootstrap.bundle.js",
-]
-
-app_include_css = [
-    "manufacturing_overview_drawer.bundle.css",
-]
+app_include_js = "manufacturing_overview.bundle.js"
 
 # include js, css files in header of web template
 # web_include_css = "/assets/manufacturing_overview/css/manufacturing_overview.css"

manufacturing_overview/manufacturing_overview/api.py

@@ -1,174 +1,129 @@
 import frappe
+from functools import reduce
 from datetime import date
-from typing import Dict, List
+
+frappe.utils.logger.set_log_level("DEBUG")
+logger = frappe.logger("manufacturing_overview",
+                       allow_site=True, file_count=10000)
 
 
-def get_due_in_days(d):
-    return (d - date.today()).days
+def getDueInDays(d):
+    delta = d - date.today()
+    return delta.days
 
 
-def format_date(d):
-    # keep your output format
-    return frappe.utils.formatdate(d, "dd.MM.yyyy")
+def getWorkOrders(soname):
+    return frappe.get_all("Work Order", filters={
+        'sales_order': soname}, fields=['name'])
 
 
-def calculate_status(item):
-    # item.qty is still the original ordered qty here (before subtracting delivered)
-    if item.delivered_qty > 0 and item.delivered_qty < item.qty:
-        return "Partially Delivered"
-    elif item.delivered_qty >= item.qty:
-        return "Fully Delivered"
-    elif item.currentWarehouseQty >= item.qty:
-        return "In Warehouse"
-    elif item.work_order_qty > 0:
-        return "To Produce"
+def getAmountInWarehouses(item_code):
+    bins = frappe.get_all("Bin", fields=["actual_qty"], filters=[
+                          ["actual_qty", ">", 0], ["item_code", "=", item_code]])
+    return reduce(lambda x, y: x + y,
+                  map(lambda b: b["actual_qty"], bins), 0)
+
+
+def shortenCustomerName(customer):
+    shortName = frappe.get_value(
+        'Customer', customer, 'short_name')
+    if shortName is not None:
+        return shortName
     else:
-        return "No Work Order"
-
-
-def fetch_sales_order_items() -> List[dict]:
-    # NOTE: your WHERE uses "status = 'To Deliver and Bill'".
-    # In your SELECT you also include `tabSales Order`.status as parentStatus,
-    # but the WHERE currently filters on "status" (ambiguous). We'll be explicit.
-    return frappe.db.sql(
-        """
-        SELECT
-            soi.name,
-            soi.item_name,
-            soi.item_group,
-            soi.qty,
-            soi.actual_qty,
-            soi.delivered_qty,
-            soi.delivery_date,
-            soi.item_code,
-            soi.work_order_qty,
-            soi.parent,
-            soi.production_plan_qty,
-            so.status AS parentStatus,
-            so.customer AS customer_name
-        FROM `tabSales Order Item` soi
-        INNER JOIN `tabSales Order` so ON soi.parent = so.name
-        WHERE
-            soi.delivered_qty < soi.qty
-            AND so.status = 'To Deliver and Bill'
-            AND soi.item_group = 'Produkte'
-        ORDER BY
-            soi.delivery_date, soi.item_code, soi.name
-        """,
-        as_dict=True,
-    )
-
-
-def fetch_customer_short_names(customers: List[str]) -> Dict[str, str]:
-    if not customers:
-        return {}
-
-    rows = frappe.get_all(
-        "Customer",
-        filters={"name": ["in", customers]},
-        fields=["name", "short_name"],
-    )
-    out = {}
-    for r in rows:
-        # fall back to full name if short_name missing/empty
-        out[r["name"]] = r.get("short_name") or r["name"]
-    return out
-
-
-def fetch_total_warehouse_qty(item_codes: List[str]) -> Dict[str, float]:
-    if not item_codes:
-        return {}
-
-    # Sum actual_qty across all bins with qty > 0 for those item codes
-    rows = frappe.db.sql(
-        """
-        SELECT
-            item_code,
-            SUM(actual_qty) AS total_qty
-        FROM `tabBin`
-        WHERE
-            actual_qty > 0
-            AND item_code IN %(item_codes)s
-        GROUP BY item_code
-        """,
-        {"item_codes": tuple(set(item_codes))},
-        as_dict=True,
-    )
-
-    return {r["item_code"]: float(r["total_qty"] or 0) for r in rows}
+        return customer
 
 
 def generateProductionOverviewCacheData():
-    salesOrderItems = fetch_sales_order_items()
+    logger.debug(f"Generating")
+    salesOrderItems = frappe.db.sql(
+        """
+        SELECT
+            `tabSales Order Item`.name,
+            `tabSales Order Item`.item_name,
+            `tabSales Order Item`.item_group,
+            `tabSales Order Item`.qty,
+            `tabSales Order Item`.actual_qty,
+            `tabSales Order Item`.delivered_qty,
+            `tabSales Order Item`.delivery_date,
+            `tabSales Order Item`.item_code,
+            `tabSales Order Item`.work_order_qty,
+            `tabSales Order Item`.parent,
+            `tabSales Order Item`.production_plan_qty,
+            `tabSales Order`.status as parentStatus
+        FROM
+            `tabSales Order Item`
+        INNER JOIN `tabSales Order`
+            ON
+                `tabSales Order Item`.parent = `tabSales Order`.name
+        WHERE
+        delivered_qty < qty and
+        status = 'To Deliver and Bill' and
+        item_group = "Produkte"
+        ORDER BY
+        delivery_date, item_code, name
+        """, as_dict=1)
 
-    if not salesOrderItems:
-        return []
+    currentWarehouseQtyList = []
 
-    # collect uniques
-    item_codes = [r["item_code"] for r in salesOrderItems if r.get("item_code")]
-    customers = [r["customer_name"] for r in salesOrderItems if r.get("customer_name")]
+    for soItem in salesOrderItems:
+        soItem.currentWarehouseQty = calculateCurrentWarehouseQty(
+            soItem.item_code, soItem.qty, currentWarehouseQtyList)
 
-    total_wh_qty = fetch_total_warehouse_qty(item_codes)
-    customer_short = fetch_customer_short_names(customers)
+        soItem.due_in = getDueInDays(soItem.delivery_date)
 
-    # Running "allocation" per item_code (your currentWarehouseQty logic, but O(n))
-    # In your old code:
-    # - first occurrence returns fetchedWarehouseQty
-    # - then subtract qty from stored qty for subsequent occurrences
-    remaining_after_current: Dict[str, float] = {}
+        soItem.customer = shortenCustomerName(frappe.get_value(
+            'Sales Order', soItem.parent, 'customer'))
 
-    for r in salesOrderItems:
-        item_code = r.get("item_code")
-        if not item_code:
-            r["totalWarehouseQty"] = 0
-            r["currentWarehouseQty"] = 0
-        else:
-            fetched = float(total_wh_qty.get(item_code, 0))
+        soItem = formatDate(soItem)
+        soItem = calculateStatus(soItem)
+        soItem.qty = soItem.qty - soItem.delivered_qty
+        soItem.link = '/app/sales-order/' + soItem.parent
 
-            if item_code in remaining_after_current:
-                # previously stored "qty" meant: remaining AFTER previous rows consumption
-                current_qty = remaining_after_current[item_code]
-            else:
-                # first time: show full fetched qty
-                current_qty = fetched
-
-            # Update remaining for next occurrence: subtract ordered qty (old behavior)
-            remaining_after_current[item_code] = current_qty - float(r.get("qty") or 0)
-
-            r["totalWarehouseQty"] = fetched
-            r["currentWarehouseQty"] = current_qty
-
-        # due_in uses original date (date object)
-        r["due_in"] = get_due_in_days(r["delivery_date"])
-
-        # customer short name
-        cust = r.get("customer_name") or ""
-        r["customer"] = customer_short.get(cust, cust)
-
-        # format delivery date string
-        r["delivery_date"] = format_date(r["delivery_date"])
-
-        # compute status BEFORE qty is reduced by delivered (same as your original)
-        r["status"] = calculate_status(frappe._dict(r))
-
-        # now update qty to remaining qty (same as your original)
-        r["qty"] = (r.get("qty") or 0) - (r.get("delivered_qty") or 0)
-
-        # link
-        r["link"] = "/app/sales-order/" + r["parent"]
-
-        # keep output compatibility: you previously returned dicts with these keys
-        # (delivered_qty, customer_name etc are extra; harmless)
     return salesOrderItems
 
 
+def calculateCurrentWarehouseQty(item_code, qty, currentWarehouseQtyList):
+    for warehouseQty in currentWarehouseQtyList:
+
+        if warehouseQty['item_code'] == item_code:
+            returnQty = warehouseQty['qty']
+            warehouseQty['qty'] = warehouseQty['qty'] - qty
+            return returnQty
+
+    fetchedWarehouseQty = getAmountInWarehouses(item_code)
+    currentWarehouseQtyList.append(
+        {'item_code': item_code, 'qty': fetchedWarehouseQty - qty})
+    return fetchedWarehouseQty
+
+
+def formatDate(item):
+    item.delivery_date = frappe.utils.formatdate(
+        item.delivery_date, 'dd.MM.yyyy')
+    return item
+
+
+def calculateStatus(item):
+    if item.delivered_qty > 0 and item.delivered_qty < item.qty:
+        item.status = 'Partially Delivered'
+    elif item.delivered_qty >= item.qty:
+        item.status = 'Fully Delivered'
+    elif item.currentWarehouseQty >= item.qty:
+        item.status = 'In Warehouse'
+    elif item.work_order_qty > 0:
+        item.status = 'To Produce'
+    else:
+        item.status = 'No Work Order'
+    return item
+
+
 @frappe.whitelist()
 def getSalesorderOverviewList():
-    cache_key = "production_overview"
-    salesOrderItems = frappe.cache().get_value(cache_key, expires=True)
+    logger.debug(f"Fetching Cache")
+    salesOrderItems = frappe.cache().get_value('production_overview')
 
     if salesOrderItems is None:
+        logger.debug(f"Cache not found")
         salesOrderItems = generateProductionOverviewCacheData()
-        frappe.cache().set_value(cache_key, salesOrderItems, expires_in_sec=60)
+        frappe.cache().set_value('production_overview', salesOrderItems, expires_in_sec=120)
 
     return salesOrderItems

manufacturing_overview/manufacturing_overview/productionplan.py

@@ -1,9 +1,9 @@
 import frappe
 from erpnext.stock.get_item_details import get_default_bom
 
-# frappe.utils.logger.set_log_level("DEBUG")
-# logger = frappe.logger("manufacturing_overview",
-#                       allow_site=True, file_count=50)
+frappe.utils.logger.set_log_level("DEBUG")
+logger = frappe.logger("manufacturing_overview",
+                       allow_site=True, file_count=50)
 
 # logger.debug(f"{current_value} + {value} = {updated_value}")
 

manufacturing_overview/public/css/manufacturing_overview_drawer.css

@@ -1,66 +0,0 @@
-#mo-drawer {
-  position: fixed;
-  top: 0;
-  right: 0;
-  height: 100vh;
-  width: 390px;
-
-  /* collapsed: show thin bar */
-  transform: translateX(calc(100% - 12px));
-  transition: transform 160ms ease;
-
-  z-index: 1050;
-}
-
-#mo-drawer:hover,
-#mo-drawer.mo-open {
-  transform: translateX(0);
-}
-
-#mo-drawer .mo-hover-bar {
-  position: absolute;
-  top: 0;
-  left: 0;
-
-  width: 12px;
-  height: 100%;
-
-  cursor: pointer;
-
-  background: var(--primary, #2563eb);
-  opacity: 0.35;
-}
-
-#mo-drawer:hover .mo-hover-bar {
-  opacity: 0.55;
-}
-
-#mo-drawer .mo-panel {
-  position: absolute;
-  top: 0;
-  left: 12px;
-
-  width: calc(100% - 12px);
-  height: 100%;
-
-  background: var(--card-bg, #fff);
-  border-left: 1px solid var(--border-color, #e5e7eb);
-  box-shadow: -10px 0 28px rgba(0, 0, 0, 0.16);
-
-  display: flex;
-  flex-direction: column;
-  overflow: hidden;
-}
-
-#mo-drawer .mo-content {
-  padding: 10px;
-  height: 100%;
-  overflow: auto;
-}
-
-/* Optional: hide on mobile */
-@media (max-width: 768px) {
-  #mo-drawer {
-    display: none;
-  }
-}

manufacturing_overview/public/drawer_bootstrap.bundle.js

@@ -1,50 +0,0 @@
-frappe.provide("manufacturing_overview.drawer");
-
-manufacturing_overview.drawer._bundle_loaded = false;
-
-manufacturing_overview.drawer.ensure_drawer = function () {
-    if (document.getElementById("mo-drawer")) return;
-
-    const el = document.createElement("div");
-    el.id = "mo-drawer";
-    el.innerHTML = `
-    <div class="mo-hover-bar" title="Manufacturing Overview"></div>
-    <div class="mo-panel">
-      <div class="mo-content">
-        <div id="mo-vue-root">
-          <div style="opacity:.7;padding:8px 2px;">Hover to load…</div>
-        </div>
-      </div>
-    </div>
-  `;
-
-    document.body.appendChild(el);
-
-    const loadBundleOnce = async () => {
-        if (manufacturing_overview.drawer._bundle_loaded) return;
-        manufacturing_overview.drawer._bundle_loaded = true;
-
-        // Lazy-load the heavy Vue bundle
-        await frappe.require("manufacturing_overview.bundle.js"); // bundle name, not a path
-
-        if (manufacturing_overview.drawer.mount_vue) {
-            manufacturing_overview.drawer.mount_vue("#mo-vue-root");
-        } else {
-            console.error(
-                "[manufacturing_overview] manufacturing_overview.bundle.js loaded, but mount function is missing."
-            );
-        }
-    };
-
-    // Load on first hover
-    el.addEventListener("mouseenter", loadBundleOnce, { once: true });
-
-    // Click toggles pinned open/close (also loads bundle)
-    el.querySelector(".mo-hover-bar").addEventListener("click", async () => {
-        el.classList.toggle("mo-open");
-        await loadBundleOnce();
-    });
-};
-
-// Ensure injection after desk bootstraps
-frappe.after_ajax(() => manufacturing_overview.drawer.ensure_drawer());

manufacturing_overview/public/js/manufacturing_overview.bundle.js

@@ -0,0 +1,4 @@
+import './manufacturing_overview_desk.vue';
+import './manufacturing_overview_page.js';
+import './manufacturing_overview_row.vue';
+import './production_plan.js';

manufacturing_overview/public/js/manufacturing_overview_desk.vue

@@ -0,0 +1,88 @@
+<template>
+  <div class="col-12 col-lg-4 layout-main-section-wrapper">
+    <div class="layout-main-section">
+      <div class="widget links-widget-box" style="height: auto">
+        <div class="widget-head">
+          <div class="widget-label">
+            <div class="widget-title"><svg class="icon  icon-lg" style="">
+                <!-- <use class="" href="#icon-file"></use> -->
+              </svg> <span class="ellipsis" title="Manufacturing OVerview">Manufacturing Overview</span>
+              <!-- <span>
+                <button class="btn btn-secondary btn-default btn-sm" data-label="Mark sent">Mark
+                  sent</button>
+              </span> -->
+            </div>
+            <div class="widget-subtitle"></div>
+          </div>
+          <div class="widget-control"></div>
+        </div>
+        <div class="widget-body">
+          <manufacturing-overview-row v-for="so in salesorderData" :key="so.name" v-bind:qty="so.qty"
+            v-bind:item_name="so.item_name" v-bind:item_code="so.item_code" v-bind:customer="so.customer"
+            v-bind:delivery_date="so.delivery_date" v-bind:status="so.status" v-bind:link="so.link"
+            v-bind:reference="so.parent" v-bind:due_in="so.due_in">
+          </manufacturing-overview-row>
+        </div>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script>
+import ManufacturingOverviewRow from "./manufacturing_overview_row.vue";
+
+export default {
+  components: {
+    ManufacturingOverviewRow,
+  },
+  data() {
+    return {
+      salesorderData: [
+        {
+          customer: "",
+          delivery_date: "",
+          link: "",
+          name: "",
+          item_name: "Loading...",
+          item_code: "",
+          qty: "",
+          sales_order: "",
+          status: "Unknown",
+          due_in: 0,
+        },
+      ],
+      timer: "",
+      origin: window.location.origin,
+      userPermissions: {},
+    };
+  },
+  created() {
+    this.fetchEventsList();
+    this.timer = setInterval(this.fetchEventsList, 60000);
+  },
+  methods: {
+    fetchEventsList() {
+      let self = this;
+      frappe.call({
+        method:
+          "manufacturing_overview.manufacturing_overview.api.getSalesorderOverviewList",
+        async: true,
+        args: {},
+        callback: function (r) {
+          if (r.message) {
+            self.salesorderData = r.message;
+          }
+        },
+      });
+    },
+    cancelAutoUpdate() {
+      clearInterval(this.timer);
+    },
+    beforeDestroy() {
+      clearInterval(this.timer);
+    },
+  },
+};
+</script>
+
+<style></style>

manufacturing_overview/public/js/manufacturing_overview_drawer/drawer_bootstrap.js

@@ -1,52 +0,0 @@
-frappe.provide("manufacturing_overview.drawer");
-
-manufacturing_overview.drawer._bundle_loaded = false;
-
-manufacturing_overview.drawer.ensure_drawer = function () {
-    if (document.getElementById("mo-drawer")) return;
-
-    const el = document.createElement("div");
-    el.id = "mo-drawer";
-    el.innerHTML = `
-    <div class="mo-hover-bar" title="Manufacturing Overview"></div>
-    <div class="mo-panel">
-      <div class="mo-content">
-        <div id="mo-vue-root">
-          <div style="opacity:.7;padding:8px 2px;">Hover to load…</div>
-        </div>
-      </div>
-    </div>
-  `;
-
-    document.body.appendChild(el);
-
-    const loadBundleOnce = async () => {
-        if (manufacturing_overview.drawer._bundle_loaded) return;
-        manufacturing_overview.drawer._bundle_loaded = true;
-
-        // This must exist at: /assets/manufacturing_overview/js/manufacturing_overview.bundle.js
-        await frappe.require("manufacturing_overview.bundle.js");
-
-        if (manufacturing_overview.drawer.mount_vue) {
-            manufacturing_overview.drawer.mount_vue("#mo-vue-root");
-        } else {
-            console.error(
-                "[manufacturing_overview] Bundle loaded but mount function missing."
-            );
-        }
-    };
-
-    // Load bundle on first hover (once)
-    el.addEventListener("mouseenter", loadBundleOnce, { once: true });
-
-    // Allow pin open/close via click on the bar
-    el.querySelector(".mo-hover-bar").addEventListener("click", async () => {
-        el.classList.toggle("mo-open");
-        await loadBundleOnce();
-    });
-};
-
-// Ensure injection after desk bootstrap
-frappe.after_ajax(() => {
-    manufacturing_overview.drawer.ensure_drawer();
-});

manufacturing_overview/public/js/manufacturing_overview_drawer/entry.js

@@ -1,14 +0,0 @@
-import { createApp } from "vue";
-import ManufacturingOverviewDesk from "./manufacturing_overview_desk.vue";
-
-manufacturing_overview.drawer.mount_vue = function (selector) {
-    const mountEl = document.querySelector(selector);
-    if (!mountEl) return;
-
-    // prevent remount
-    if (mountEl.__mo_app__) return;
-
-    const app = createApp(ManufacturingOverviewDesk, {});
-    mountEl.__mo_app__ = app;
-    app.mount(mountEl);
-};

manufacturing_overview/public/js/manufacturing_overview_drawer/manufacturing_overview_desk.vue

@@ -1,84 +0,0 @@
-<template>
-  <div class="widget links-widget-box" style="height: auto">
-    <div class="widget-head">
-      <div class="widget-label">
-        <div class="widget-title">
-          <span class="ellipsis" title="Manufacturing Overview">
-            Manufacturing Overview
-          </span>
-        </div>
-        <div class="widget-subtitle"></div>
-      </div>
-      <div class="widget-control"></div>
-    </div>
-
-    <div class="widget-body">
-      <manufacturing-overview-row
-        v-for="so in salesorderData"
-        :key="so.name"
-        :qty="so.qty"
-        :item_name="so.item_name"
-        :item_code="so.item_code"
-        :customer="so.customer"
-        :delivery_date="so.delivery_date"
-        :status="so.status"
-        :link="so.link"
-        :reference="so.parent"
-        :due_in="so.due_in"
-      />
-    </div>
-  </div>
-</template>
-
-<script>
-import ManufacturingOverviewRow from "./manufacturing_overview_row.vue";
-
-export default {
-  components: { ManufacturingOverviewRow },
-  data() {
-    return {
-      salesorderData: [
-        {
-          customer: "",
-          delivery_date: "",
-          link: "",
-          name: "",
-          item_name: "Loading...",
-          item_code: "",
-          qty: "",
-          sales_order: "",
-          status: "Unknown",
-          due_in: 0,
-        },
-      ],
-      timer: null,
-    };
-  },
-  created() {
-    this.fetchEventsList();
-    this.timer = setInterval(this.fetchEventsList, 10000);
-  },
-  beforeUnmount() {
-    clearInterval(this.timer);
-  },
-  methods: {
-    fetchEventsList() {
-      frappe.call({
-        method:
-          "manufacturing_overview.manufacturing_overview.api.getSalesorderOverviewList",
-        async: true,
-        args: {},
-        callback: (r) => {
-          if (r.message) this.salesorderData = r.message;
-        },
-      });
-    },
-  },
-};
-</script>
-
-<style scoped>
-.widget-body {
-  padding-top: 6px;
-}
-</style>

manufacturing_overview/public/js/manufacturing_overview_page.js

@@ -0,0 +1,11 @@
+import ManufacturingOverviewDesk from "./manufacturing_overview_desk.vue";
+
+$(document).ready(function () {
+    $(".layout-main-section-wrapper").after('<div class="col-12 col-lg-4 layout-main-section-wrapper" id="manufacturing-overview-body"></div>');
+    var pod = new Vue({
+        el: "#manufacturing-overview-body",
+        render(h) {
+            return h(ManufacturingOverviewDesk, {});
+        }
+    });
+});

manufacturing_overview/public/js/manufacturing_overview_row.vue

@@ -1,41 +1,28 @@
 <template>
-  <a @click="pushRoute(link)" class="row link-item text-wrap ellipsis" role="link">
+  <a @click="pushRoute(link)" class="row link-item text-wrap ellipsis onbpoard-spotlight" type="Link">
     <div class="col col-xs-8">
-      <span
-        class="indicator-pill no-margin"
-        :class="{
+      <span class="indicator-pill no-margin" v-bind:class="{
         red: status === 'No Work Order',
         blue: status === 'Partially Delivered',
         gray: status === 'Fully Delivered',
         green: status === 'In Warehouse',
         yellow: status === 'To Produce',
         grey: status === 'Unknown',
-        }"
-      ></span>
-
+      }"></span>
       <span class="widget-subtitle">{{ qty }}</span> -
       <span class="widget-title">{{ item_name }}</span>
-
       <div>
-        <small v-if="customer && item_code" class="color-secondary">
-          {{ customer }} -
-          <a
-            @click.stop.prevent="pushRoute('/desk/item/' + item_code)"
-            role="link"
-          >
-            {{ item_code }}
-          </a>
-        </small>
-
-        <small v-else-if="customer" class="color-secondary">
-          {{ customer }}
-        </small>
-
-        <small v-else-if="item_code" class="color-secondary">
-          {{ item_code }}
-        </small>
+        <small v-if="customer && item_code" class="color-secondary">{{ customer }} -
+          <a @click="pushRoute('/app/item/' + item_code)" type="Link">{{
+              item_code
+          }}</a></small>
+        <small v-else-if="customer" class="color-secondary">{{
+            customer
+        }}</small>
+        <small v-else-if="item_code" class="color-secondary">{{
+            item_code
+        }}</small>
       </div>
-
       <div>
         <small class="color-secondary">{{ reference }}</small>
       </div>
@@ -44,11 +31,9 @@
     <div v-if="due_in < 0" class="text-muted ellipsis color-secondary col col-xs-4 text-right">
       <b style="color: red">{{ delivery_date }}</b>
     </div>
-
     <div v-else-if="due_in === 0" class="text-muted ellipsis color-secondary col col-xs-4 text-right">
       <b style="color: black">{{ delivery_date }}</b>
     </div>
-
     <div v-else class="text-muted ellipsis color-secondary col col-xs-4 text-right">
       {{ delivery_date }}
     </div>
@@ -77,4 +62,6 @@ export default {
 };
 </script>
 
-<style scoped></style>
+<style>
+
+</style>

manufacturing_overview/public/js/production_plan.js

@@ -0,0 +1,31 @@
+frappe.ui.form.on("Sales Order", {
+    refresh: function (frm) {
+        frm.add_custom_button(__('Production Plan'), function () {
+            frappe.call({
+                method: 'manufacturing_overview.manufacturing_overview.productionplan.makepp',
+                args: {
+                    sales_order: frm.docname
+                },
+                callback: function (r) {
+                    if (r.message.status === 201) {
+                        frappe.msgprint({
+                            title: __('Created'),
+                            indicator: 'green',
+                            message: __('Production Plan {0} created.',
+                                ['<a href="/app/production-plan/' + r.message.docname + '">' + r.message.docname + '</a>'])
+                        });
+                    } else if (r.message.status === 400) {
+                        frappe.msgprint({
+                            title: __('Already exists'),
+                            indicator: 'yellow',
+                            message: __('Production Plan {0} already exists.',
+                                ['<a href="/app/production-plan/' + r.message.docname + '">' + r.message.docname + '</a>'])
+                        });
+                    } else {
+                        console.log(r.message)
+                    }
+                }
+            });
+        })
+    }
+});

manufacturing_overview/public/manufacturing_overview.bundle.js

@@ -1,19 +0,0 @@
-import { createApp } from "vue";
-import ManufacturingOverviewDesk from "./js/manufacturing_overview_drawer/manufacturing_overview_desk.vue";
-
-frappe.provide("manufacturing_overview.drawer");
-
-/**
- * Called by drawer_bootstrap.bundle.js after lazy-loading this bundle
- */
-manufacturing_overview.drawer.mount_vue = function (selector = "#mo-vue-root") {
-    const el = document.querySelector(selector);
-    if (!el) return;
-
-    // Prevent double mounting if user hovers multiple times
-    if (el.__mo_vue_app__) return;
-
-    const app = createApp(ManufacturingOverviewDesk);
-    el.__mo_vue_app__ = app;
-    app.mount(el);
-};

manufacturing_overview/public/manufacturing_overview_drawer.bundle.css

@@ -1,66 +0,0 @@
-#mo-drawer {
-  position: fixed;
-  top: 0;
-  right: 0;
-  height: 100vh;
-  width: 390px;
-
-  /* collapsed: show thin bar */
-  transform: translateX(calc(100% - 12px));
-  transition: transform 160ms ease;
-
-  z-index: 1050;
-}
-
-#mo-drawer:hover,
-#mo-drawer.mo-open {
-  transform: translateX(0);
-}
-
-#mo-drawer .mo-hover-bar {
-  position: absolute;
-  top: 0;
-  left: 0;
-
-  width: 12px;
-  height: 100%;
-
-  cursor: pointer;
-
-  background: var(--primary, #2563eb);
-  opacity: 0.35;
-}
-
-#mo-drawer:hover .mo-hover-bar {
-  opacity: 0.55;
-}
-
-#mo-drawer .mo-panel {
-  position: absolute;
-  top: 0;
-  left: 12px;
-
-  width: calc(100% - 12px);
-  height: 100%;
-
-  background: var(--card-bg, #fff);
-  border-left: 1px solid var(--border-color, #e5e7eb);
-  box-shadow: -10px 0 28px rgba(0, 0, 0, 0.16);
-
-  display: flex;
-  flex-direction: column;
-  overflow: hidden;
-}
-
-#mo-drawer .mo-content {
-  padding: 10px;
-  height: 100%;
-  overflow: auto;
-}
-
-/* Optional: hide on mobile */
-@media (max-width: 768px) {
-  #mo-drawer {
-    display: none;
-  }
-}