Skip to content

[FIX] hr: hide new contract button if no start date#5186

Open
LucasDaniOV wants to merge 1 commit into
master-hr-onboarding-luoudfrom
master-hr-onboarding-new-contract-button-invisible-luoud
Open

[FIX] hr: hide new contract button if no start date#5186
LucasDaniOV wants to merge 1 commit into
master-hr-onboarding-luoudfrom
master-hr-onboarding-new-contract-button-invisible-luoud

Conversation

@LucasDaniOV

@LucasDaniOV LucasDaniOV commented Jun 23, 2026

Copy link
Copy Markdown

This hides the "New Contract" button if there is not an existing start date

Task: 6326361

@robodoo

robodoo commented Jun 23, 2026

Copy link
Copy Markdown

This PR targets the un-managed branch odoo-dev/odoo:master-hr-onboarding-luoud, it needs to be retargeted before it can be merged.

@arpi-odoo arpi-odoo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code is good but the PR description should be the same as the commit message. You can always delete the automatic part that's added by github, it is only there for external people that want to contribute to odoo since the odoo repo is open source.

@LucasDaniOV LucasDaniOV requested a review from arpi-odoo June 23, 2026 15:05
@arpi-odoo

Copy link
Copy Markdown

@LucasDaniOV I meant this part 😛

This hides the "New Contract" button if there is not an existing start date

Task: 6326361

Also, the tag in the commit and PR title is not correct, this is a FIX.

@LucasDaniOV LucasDaniOV changed the title [IMP] hr: hide new contract button if no start date [FIX] hr: hide new contract button if no start date Jun 23, 2026
This hides the "New Contract" button if there is not an existing start date

Task: 6326361
@LucasDaniOV LucasDaniOV force-pushed the master-hr-onboarding-new-contract-button-invisible-luoud branch from f9ab1be to 8496fd4 Compare June 23, 2026 15:17
@LucasDaniOV

Copy link
Copy Markdown
Author

@arpi-odoo should be good now

@arpi-odoo arpi-odoo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants