[IMP] hr_skills: unifying certification reports

- Removed default group by from Learning > Certifications to be more generice
- Removed certifications from Reporting > Skills to reduce redundancy

task-5385053

Author: hazemayman1

addons/hr_skills/report/hr_employee_certification_report_views.xml

@@ -61,11 +61,4 @@
             </p>
         </field>
     </record>
-
-    <menuitem
-        id="hr_employee_certification_report_menu"
-        name="Certifications"
-        action="hr_employee_certification_report_action"
-        parent="hr_skills.hr_employee_skill_report_menu"
-        sequence="25"/>
 </odoo>

addons/hr_skills/report/hr_employee_skill_report_views.xml

@@ -89,6 +89,6 @@
         id="hr_employee_skill_inventory_report_menu"
         name="Skills Inventory"
         action="hr_employee_skill_report_action"
-        parent="hr_skills.hr_employee_skill_report_menu"
+        parent="hr.hr_menu_hr_reports"
         sequence="15"/>
 </odoo>

addons/hr_skills/views/hr_views.xml

@@ -594,7 +594,7 @@
         <field name="res_model">hr.employee.skill</field>
         <field name="path">certifications</field>
         <field name="domain">[('is_certification', '=', True), ('company_id', 'in', allowed_company_ids)]</field>
-        <field name="context">{'show_employee': True, 'search_default_group_by_type': 1}</field>
+        <field name="context">{'show_employee': True}</field>
         <field name="view_mode">list,form</field>
         <field name="view_ids" eval="[(5, 0, 0),
             (0, 0, {'view_mode': 'list', 'view_id': False}),
@@ -609,12 +609,6 @@
         sequence="7"
         groups="hr.group_hr_user"/>
 
-    <menuitem
-    id="hr_employee_skill_report_menu"
-    name="Skills"
-    parent="hr.hr_menu_hr_reports"
-    sequence="15"/>
-
     <menuitem
         id="hr_skill_learning_menu"
         name="Learning"