Skip to content

Conversation

@Kimkhoi3010
Copy link

@Kimkhoi3010 Kimkhoi3010 commented Jan 19, 2026

Context:

Fix various ESLint warnings in website sale JS modules:

  • website_sale_product_attribute_value_filter_existing
  • website_sale_acquirer_confirm_order
  • website_sale_checkout_skip_payment
  • website_sale_stock_provisioning_date
  • website_sale_require_legal
  • website_snippet_product_category
  • website_sale_secondary_unit

Changes

  • website_sale_product_attribute_value_filter_existing: Adjust handling of the Odoo-specific module header to satisfy JSDoc / ESLint expectations in website_sale_product_attribute_value_filter_existing_search_desk_tour.esm.js.
  • website_sale_acquirer_confirm_order: Reorder imports so that import * as comes before named imports, complying with the sort-imports rule in website_sale_acquirer_confirm_order_tour.esm.js.
  • website_sale_checkout_skip_payment: Reorder imports so that import * as comes before named imports to satisfy sort-imports in website_sale_checkout_skip_payment_tour.esm.js.
  • website_sale_stock_provisioning_date: Sort named imports alphabetically by module path to fix sort-imports warnings in website_sale_stock_provisioning_date_tour.esm.js.
  • website_sale_require_legal: Reorder imports so that import * as comes before named imports, complying with the sort-imports rule in tour.esm.js.
  • website_snippet_product_category: Sort imports alphabetically by module path to satisfy sort-imports in frontend.esm.js.
  • website_sale_secondary_unit: Refactor _onChangeSecondaryUom to avoid reassigning the ev parameter by introducing a local eventToUse variable, keeping the existing fake-event hack while satisfying the no-param-reassign rule in website_sale_secondary_unit.esm.js.

@OCA-git-bot
Copy link
Contributor

Hi @Tardo, @pilarvargas-tecnativa,
some modules you are maintaining are being modified, check this out!

Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants