Skip to content

[FIX] developer/tutorials: remove remark for present fields in views#19074

Open
YassinWalid wants to merge 1 commit into
18.0from
18.0-fix_mandatory_view_field_remark-yabd
Open

[FIX] developer/tutorials: remove remark for present fields in views#19074
YassinWalid wants to merge 1 commit into
18.0from
18.0-fix_mandatory_view_field_remark-yabd

Conversation

@YassinWalid

Copy link
Copy Markdown

After odoo/odoo#137031, the views no longer require the fields used in them to be present in the view, as they are automatically added to the view by the framework. This commit removes the outdated remark in the tutorial that suggested otherwise.

@robodoo

robodoo commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

@C3POdoo
C3POdoo requested a review from a team July 23, 2026 09:55

@AntoineVDV AntoineVDV left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@robodoo delegate+

Note that it's recommended to reference commits, and not PRs, in your commit message (in the PR description it's fine). That is because each of your or the mergebot's pushes will log a mention on the target PR, which quickly becomes annoying spam.

@YassinWalid
YassinWalid force-pushed the 18.0-fix_mandatory_view_field_remark-yabd branch 3 times, most recently from aecfb00 to 0a69580 Compare July 23, 2026 11:45
After the odoo/odoo@6f06420, the views
no longer require the fields used in them to be present in the view, as
they are automatically added to the view by the framework. This commit
removes the outdated remark in the tutorial that suggested otherwise.
@YassinWalid
YassinWalid force-pushed the 18.0-fix_mandatory_view_field_remark-yabd branch from 0a69580 to 7308e55 Compare July 23, 2026 12:03
@YassinWalid

Copy link
Copy Markdown
Author

@robodoo r+

robodoo pushed a commit that referenced this pull request Jul 23, 2026
After the odoo/odoo@6f06420, the views
no longer require the fields used in them to be present in the view, as
they are automatically added to the view by the framework. This commit
removes the outdated remark in the tutorial that suggested otherwise.

closes #19074

Signed-off-by: Yassin Walid Abdelkarim (yabd) <yabd@odoo.com>
robodoo pushed a commit that referenced this pull request Jul 23, 2026
After the odoo/odoo@6f06420, the views
no longer require the fields used in them to be present in the view, as
they are automatically added to the view by the framework. This commit
removes the outdated remark in the tutorial that suggested otherwise.

closes #19074

Signed-off-by: Yassin Walid Abdelkarim (yabd) <yabd@odoo.com>
robodoo pushed a commit that referenced this pull request Jul 23, 2026
After the odoo/odoo@6f06420, the views
no longer require the fields used in them to be present in the view, as
they are automatically added to the view by the framework. This commit
removes the outdated remark in the tutorial that suggested otherwise.

closes #19074

Signed-off-by: Yassin Walid Abdelkarim (yabd) <yabd@odoo.com>
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.

4 participants