diff --git a/yuthon_hr_attendance/models/go_approval.py b/yuthon_hr_attendance/models/go_approval.py
index 07e9f8930..4ca4cdf70 100644
--- a/yuthon_hr_attendance/models/go_approval.py
+++ b/yuthon_hr_attendance/models/go_approval.py
@@ -39,13 +39,19 @@ class GoApprovals(models.Model):
     work_end = fields.Boolean(string="结束")
     is_day = fields.Boolean(string="按天")
     is_hour = fields.Boolean(string="按小时")
-    start_day_selection = fields.Selection([('half', '上半天'),('afternoon', '下半天')], string="天", required=True)
-    end_day_selection = fields.Selection([('half', '上半天'), ('afternoon', '下半天')], string="天", required=True)
+    start_day_selection = fields.Selection([('half', '上半天'),('afternoon', '下半天')], string="天")
+    end_day_selection = fields.Selection([('half', '上半天'), ('afternoon', '下半天')], string="天")
     start_date = fields.Date(string="开始日期", default=fields.Date.today)
     end_date = fields.Date(string="结束日期", default=fields.Date.today)
     is_this_department = fields.Boolean(string='是否是本部门', default=True)
     department_ids = fields.Many2many('hr.department', string="多部门")
 
+    @api.constrains('is_day', 'start_day_selection', 'end_day_selection')
+    def _check_selection(self):
+        for rec in self:
+            if rec.is_day and (not rec.start_day_selection or not rec.end_day_selection):
+                raise UserError("勾选按天后，必须选择开始时段和结束时段！")
+
     def _get_default_name_title(self):
         return self._description if self._description else self._name
 
diff --git a/yuthon_hr_attendance/views/go_approval.xml b/yuthon_hr_attendance/views/go_approval.xml
index 7721e9d5d..3aec47b57 100644
--- a/yuthon_hr_attendance/views/go_approval.xml
+++ b/yuthon_hr_attendance/views/go_approval.xml
@@ -68,8 +68,9 @@
                                 <field name="end_date" invisible="not is_day"/>
                             </group>
                             <group>
-                                <field name="start_day_selection" invisible="not is_day" string=" "/>
-                                <field name="end_day_selection" invisible="not is_day" string=" "/>
+                                <field name="is_day" />
+                                <field name="start_day_selection" required="is_day" />
+                                <field name="end_day_selection" required="is_day" />
                             </group>
                             <group/>
                         </group>