From 749382a1e742912774ae67f07799fda579ca8c4e Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@odoo.com>
Date: Thu, 28 Nov 2024 13:22:32 +0100
Subject: [PATCH] [REM] runbot_merge: freeze wizard auto-refresh

Turns out to not work well in 17.0, and after consideration moc hasn't
really used the auto-update feature (or the required-prs gate in
general to be honest), usually he knows what PRs he's waiting for and
only validates once he's confirmed every which way.

So it's probably not worth fixing the thing. According to jpp, this
should probably use something based on bus subscriptions to update
just the field (though tbf the `root.update` call doesn't really seem
to be "deep" anymore, so in reality rather than update the *form*'s
record I should probably have tried reloading the required_pr_ids
records to fetch the new color).

Closes #997
---
 runbot_merge/__manifest__.py                  |  1 -
 runbot_merge/models/project_freeze/views.xml  |  2 +-
 .../static/src/project_freeze/index.js        | 36 -------------------
 3 files changed, 1 insertion(+), 38 deletions(-)
 delete mode 100644 runbot_merge/static/src/project_freeze/index.js

diff --git a/runbot_merge/__manifest__.py b/runbot_merge/__manifest__.py
index b7e69773..65dfc4b4 100644
--- a/runbot_merge/__manifest__.py
+++ b/runbot_merge/__manifest__.py
@@ -30,7 +30,6 @@
         ],
         'web.assets_backend': [
             'runbot_merge/static/scss/runbot_merge_backend.scss',
-            'runbot_merge/static/src/project_freeze/index.js',
         ],
     },
     'post_load': 'enable_sentry',
diff --git a/runbot_merge/models/project_freeze/views.xml b/runbot_merge/models/project_freeze/views.xml
index d243edcd..bc26008e 100644
--- a/runbot_merge/models/project_freeze/views.xml
+++ b/runbot_merge/models/project_freeze/views.xml
@@ -3,7 +3,7 @@
         <field name="name">Freeze Wizard Configuration Screen</field>
         <field name="model">runbot_merge.project.freeze</field>
         <field name="arch" type="xml">
-            <form js_class="freeze_wizard">
+            <form>
                 <sheet>
                     <div class="alert alert-warning" role="alert"
                          invisible="not errors">
diff --git a/runbot_merge/static/src/project_freeze/index.js b/runbot_merge/static/src/project_freeze/index.js
deleted file mode 100644
index 8dca49bc..00000000
--- a/runbot_merge/static/src/project_freeze/index.js
+++ /dev/null
@@ -1,36 +0,0 @@
-/** @odoo-module */
-
-import {useEffect} from "@odoo/owl";
-import {x2ManyCommands} from "@web/core/orm_service";
-import {registry} from "@web/core/registry";
-import {FormController} from "@web/views/form/form_controller";
-import {formView} from "@web/views/form/form_view";
-
-class FreezeController extends FormController {
-    setup() {
-        super.setup();
-
-        useEffect(() => {
-            const interval = setInterval(async () => {
-                const root = this.model.root;
-                if (await root.isDirty()) {
-                    root.update({
-                        required_pr_ids: x2ManyCommands.set(
-                            root.data.required_pr_ids.currentIds,
-                        ),
-                    });
-                } else {
-                    root.load();
-                }
-            }, 1000);
-            return () => {
-                clearInterval(interval);
-            };
-        }, () => []);
-    }
-}
-
-registry.category("views").add('freeze_wizard', {
-    ...formView,
-    Controller: FreezeController,
-});