Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIG] [18.0] hr_grade_rank: migrated module #656

Open
wants to merge 3 commits into
base: 18.0
Choose a base branch
from

Conversation

ShwetaSerpentCS
Copy link
Contributor

@ShwetaSerpentCS ShwetaSerpentCS commented Oct 23, 2024

Responsible : Jayram

@ketanserpentcs
Copy link

Testing is assigned to Pooja

</sheet>
</form>
</field>
</record>

<!-- GRADE TREE VIEW -->
<record id="view_employee_grade_tree" model="ir.ui.view">
<field name="name">grade.grade.tree</field>
<record id="view_employee_grade_list" model="ir.ui.view">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to replace tree with list in the ID of any view. as it may cause issues if define reference of the view based on ID.

</sheet>
</form>
</field>
</record>

<!-- RANK TREE VIEW -->
<record id="view_employee_rank_tree" model="ir.ui.view">
<field name="name">rank.rank.tree</field>
<record id="view_employee_rank_list" model="ir.ui.view">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above, do not change the view ID

</field>
</field>
</record>

<!-- EMPLOYEES INHERITED TREE VIEW -->
<record id="view_employee_tree_grade_rank" model="ir.ui.view">
<field name="name">hr.employee.tree.grade.rank</field>
<record id="view_employee_list_grade_rank" model="ir.ui.view">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

@Trivedi-Vacha-SerpentCS Trivedi-Vacha-SerpentCS added the In QA Under testing by QA team label Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In QA Under testing by QA team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants