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

[IMP] hr_contract: add sheet to contact form and simplify form #27333

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@Yenthe666
Contributor

Yenthe666 commented Oct 1, 2018

Description of the issue/feature this PR addresses: Not perfectly created view with not used element settings

Current behavior before PR: The contract type form view has no <sheet> set.

Desired behavior after PR is merged: The contract type form view has a <sheet> set and doesn't have the col elements set as the view renders identically without them.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@tivisse tivisse self-assigned this Oct 17, 2018

@tivisse tivisse added HR and removed HR labels Nov 8, 2018

@robodoo robodoo removed the CI 🤖 label Nov 8, 2018

[IMP] hr_contract: add sheet on contact type form
It's better to display form views in sheets.
@tivisse

This comment has been minimized.

Contributor

tivisse commented Nov 8, 2018

@robodoo robodoo added the r+ 👌 label Nov 8, 2018

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