Skip to content

[ADD][stock_ux] import excel file#938

Closed
jcadhoc wants to merge 2 commits into
ingadhoc:19.0from
adhoc-dev:19.0-t-65009-jc
Closed

[ADD][stock_ux] import excel file#938
jcadhoc wants to merge 2 commits into
ingadhoc:19.0from
adhoc-dev:19.0-t-65009-jc

Conversation

@jcadhoc

@jcadhoc jcadhoc commented May 28, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 28, 2026 17:53
@roboadhoc

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds an import template for products in the stock_ux module, allowing users to download a pre-formatted XLSX template when importing products from the product list view.

Changes:

  • Override get_import_templates on product.template to return a custom XLSX template when the stock_product_template context key is set.
  • Extend the stock.product_template_action_product action context to include stock_product_template: 1.
  • Bump module version to 19.0.1.9.0.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
stock_ux/models/product_template.py Adds get_import_templates override returning a custom XLSX template based on context.
stock_ux/views/product_template_views.xml Adds context key to the product action to trigger the custom import template.
stock_ux/manifest.py Version bump.

</record>

<record id="stock.product_template_action_product" model="ir.actions.act_window">
<field name="context">{"search_default_goods": 1, "default_is_storable": True, "stock_product_template": 1}</field>
@jcadhoc jcadhoc changed the title [ADD] import excel file [ADD][stock_ux] import excel file Jun 3, 2026
@mav-adhoc

Copy link
Copy Markdown
Contributor

@roboadhoc r+ nobump

@roboadhoc

Copy link
Copy Markdown
Collaborator

@jcadhoc @mav-adhoc because this PR has multiple commits, I need to know how to merge it:

  • merge to merge directly, using the PR as merge commit message
  • rebase-merge to rebase and merge, using the PR as merge commit message
  • rebase-ff to rebase and fast-forward

@mav-adhoc

Copy link
Copy Markdown
Contributor

@roboadhoc rebase-ff

@roboadhoc

Copy link
Copy Markdown
Collaborator

Merge method set to rebase and fast-forward.

roboadhoc pushed a commit that referenced this pull request Jun 3, 2026
Part-of: #938
Signed-off-by: Matias Velazquez <mav@adhoc.com.ar>
@roboadhoc roboadhoc closed this in eb84e99 Jun 3, 2026
@roboadhoc roboadhoc deleted the 19.0-t-65009-jc branch June 3, 2026 16:08
@roboadhoc roboadhoc added the 18.1 label Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants