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] fleet: add sheet tag to form views #27327

Closed
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 views.

Current behavior before PR: Multiple base views have no <sheet> set.

Desired behavior after PR is merged: The form views have a <sheet> set

--
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

[IMP] fleet: add sheet on model brand and tag form views
It's better to use sheet on form views

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

@tivisse

This comment has been minimized.

Contributor

tivisse commented Nov 8, 2018

robodoo pushed a commit that referenced this pull request Nov 8, 2018

[IMP] fleet: add sheet on model brand and tag form views
It's better to use sheet on form views

closes #27327
@robodoo

This comment has been minimized.

Contributor

robodoo commented Nov 8, 2018

Merged, thanks!

@robodoo robodoo closed this Nov 8, 2018

@Yenthe666

This comment has been minimized.

Contributor

Yenthe666 commented Nov 9, 2018

Thanks a lot!

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