From 3c4765f54cac1b91f463b760e05f3c14b3b15b3c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=ADctor=20Mart=C3=ADnez?=
 <victor.martinez@tecnativa.com>
Date: Wed, 9 Dec 2020 12:47:06 +0100
Subject: [PATCH] [ADD] document_page: Add required fields to improve
 page_history

---
 document_page/demo/document_page.xml          | 6 ++++++
 document_page/readme/CONTRIBUTORS.rst         | 1 +
 document_page/views/document_page.xml         | 8 +++++++-
 document_page/views/document_page_history.xml | 4 ++--
 4 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/document_page/demo/document_page.xml b/document_page/demo/document_page.xml
index 446da983..36e494bf 100644
--- a/document_page/demo/document_page.xml
+++ b/document_page/demo/document_page.xml
@@ -19,6 +19,8 @@ Additional ressources
     </record>
     <record id="demo_page1" model="document.page">
         <field name="name">Odoo 13.0 Functional Demo</field>
+        <field name="draft_name">1.0</field>
+        <field name="draft_summary">Init</field>
         <field name="parent_id" ref="demo_category1" />
         <field name="content">
 <![CDATA[
@@ -48,6 +50,8 @@ company, with your clients and implement it now for your business.<br>
     <record id="demo_page2" model="document.page">
         <field name="name">Personalise Dashboards</field>
         <field name="parent_id" ref="demo_category1" />
+        <field name="draft_name">1.0</field>
+        <field name="draft_summary">Init</field>
         <field name="content">
 <![CDATA[
 <br>
@@ -87,6 +91,8 @@ you change your mind there is a reset button to return to the default view.<br>
     <record id="demo_page3" model="document.page">
         <field name="name">Touchscreen Point of Sale</field>
         <field name="parent_id" ref="demo_category1" />
+        <field name="draft_name">1.0</field>
+        <field name="draft_summary">Init</field>
         <field name="content">
 <![CDATA[
 <br>
diff --git a/document_page/readme/CONTRIBUTORS.rst b/document_page/readme/CONTRIBUTORS.rst
index 8bfc24df..4ba67b75 100644
--- a/document_page/readme/CONTRIBUTORS.rst
+++ b/document_page/readme/CONTRIBUTORS.rst
@@ -7,3 +7,4 @@
 * `Tecnativa <https://www.tecnativa.com>`_:
 
   * Ernesto Tejeda
+  * Víctor Martínez
diff --git a/document_page/views/document_page.xml b/document_page/views/document_page.xml
index 1870ba2b..52dc9f40 100644
--- a/document_page/views/document_page.xml
+++ b/document_page/views/document_page.xml
@@ -65,6 +65,7 @@
                                 <group>
                                     <field
                                         name="parent_id"
+                                        required="True"
                                         string="Category"
                                         context="{'default_type':'category'}"
                                     />
@@ -81,10 +82,15 @@
                                     />
                                 </group>
                                 <group string="Revision">
-                                    <field name="draft_name" placeholder="Rev 01" />
+                                    <field
+                                        name="draft_name"
+                                        placeholder="Rev 01"
+                                        required="True"
+                                    />
                                     <field
                                         name="draft_summary"
                                         placeholder="eg: Changed ... for ..."
+                                        required="True"
                                     />
                                 </group>
                             </group>
diff --git a/document_page/views/document_page_history.xml b/document_page/views/document_page_history.xml
index 1fdf960b..00047b6e 100644
--- a/document_page/views/document_page_history.xml
+++ b/document_page/views/document_page_history.xml
@@ -21,8 +21,8 @@
         <field name="model">document.page.history</field>
         <field name="arch" type="xml">
             <search string="Document Page History">
-                <field name="page_id" />
-                <field name="content" />
+                <field name="page_id" required="True" />
+                <field name="content" required="True" />
                 <field name="create_uid" />
                 <group expand="0" string="Group By...">
                     <filter