Skip to content

[FIX] booking_channex: fix model type in server actions#4

Open
abdrahmanrashed wants to merge 137 commits intosaas-19.1-booking-channex-setting-chgafrom
saas-19.1-booking-channex-mapping-aras
Open

[FIX] booking_channex: fix model type in server actions#4
abdrahmanrashed wants to merge 137 commits intosaas-19.1-booking-channex-setting-chgafrom
saas-19.1-booking-channex-mapping-aras

Conversation

@abdrahmanrashed
Copy link
Copy Markdown

  • Change x_model_type values in channex server actions to match selection values.
  • Minor refactoring.

task-5913987

vava-odoo and others added 30 commits December 22, 2025 10:40
Same as excise management features, deposit management is useful in
several industries. It therefore makes sense to package it in a
dedicated submodule. This sets a clear and meaningful distinction
between excise, deposit and industry-related features. It also reduces
the duplicated records in industry modules by relying on the new
deposit_management module.

task-5062926

closes odoo#1462

X-original-commit: 04e7943
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
X-original-commit: 1fd956a
Part-of: odoo#1464
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
X-original-commit: 6f595d7
Part-of: odoo#1464
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
X-original-commit: 18e7846
Part-of: odoo#1464
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
closes odoo#1464

X-original-commit: a15bd2f
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
…nowledge article of NGO and the website page /about-us.

closes odoo#1468

X-original-commit: 77fb33a
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Before this commit, it was impossible to create a new contact, since
the on_change on the ID scan was triggered for no reason. This
commit adds a filter_domain so that the automation is only triggered
if the ID scan is updated. This also fix the linked server action, which
was wrongly using record instead of partner in the loop.

Task-5440779

closes odoo#1470

Forward-port-of: odoo#1467
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
- Removed all `product.template.attribute.value` records as they are automatically created on `product.template.attribute.line` creation and can no longer be overridden due to new key uniqueness contraint emerging from `product_template_attribute_excluded_value_ids_rel`: odoo/odoo#220378

- Replaced `company_type` and `is_company`  fields with 'vat` fields since company type is now computed: odoo/odoo#211043

- Removed `appointment_progress_bar` and `appointment_info` header as they are removed from standard: odoo/enterprise#101785

- Updated restaurant records to match new customization options: https://github.com/odoo/odoo/pull/239681/files

- Additional minor view fixes to match standard  view modifications.

task-5417814

closes odoo#1457

Signed-off-by: Pratima Gupta (pgu) <pgu@odoo.com>
We updated the PO files by taking the version from 19.0 and `msgmerge`
them with the latest POT file.

closes odoo#1460

Signed-off-by: Dylan Kiss (dyki) <dyki@odoo.com>
- Removing `version: 1.0` since it is now implicitly set as default: odoo/odoo@7176195

Part-of: odoo#1477
Signed-off-by: Abdelrahman Rashed (aras) <aras@odoo.com>
- Removed the remaining `product.templated.attribute.value` models that weren't removed in: odoo#1473

-  Modified `real_estate` view according to changes in: odoo/odoo@6ffc28f

-  Modified `real_estate` website template according to changes in: odoo/odoo@f4aa56f

closes odoo#1477

Signed-off-by: Abdelrahman Rashed (aras) <aras@odoo.com>
…eference

Fixes the ValueError when duplicating analytical accounts after the analytic_plan_properties record has been deleted.

The x_is_property computed field was failing with:
`ValueError: External ID not found in the system: industry_real_estate.analytic_plan_properties`

**Changes:**
- Added `raise_if_not_found=False` parameter to `env.ref()` call
- Updated field compute logic to handle missing reference gracefully
- Bumped module version to 1.1

closes odoo#1443

closes odoo#1485

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
This commit fixes industries that call _create_journal_and_payment_methods now that pos_online_payment adds default demo data
odoo/odoo@68fc7e8
Removes the call and replace `<field name="payment_method_ids"/>`'s eval with `[Command.link(ref('id_of_payment_method'))]`

closes odoo#1448

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
In this commit we have removed the titles which are not aligned with our website.

Task-5461812

closes odoo#1493

X-original-commit: 78708a6
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
In order to automate the export of POT files in industry modules, we
add a test that exports the source terms for every installed industry
module and saves it to the appropriate location.

This can be used by runbot and our sync module to automate the process.

closes odoo#1500

X-original-commit: 4b008c8
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Dylan Kiss (dyki) <dyki@odoo.com>
It appears that we expose all the complexity of the setup of product planning shifts to our users while we may be able to hide part of the complexity for simpler setup of their stay offers and associated resources.

In this commit, we aim to:
- Simplifying the product form when a stay offer.
- Using consistent and simpler naming.
- Automate planning role creation/deletion to bind the stay offers to resources to simplify the experience of users.

task-5367647

closes odoo#1491

X-original-commit: 7df848e
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Abdelrahman Rashed (aras) <aras@odoo.com>
Before this commit the maintenance contract logic relied on a hard-coded product
ID. As a result, it only worked when that specific maintenance product was
selected. Creating a new maintenance product with the same configuration caused
the maintenance contract flow to fail.

After this commit it removes the static dependency and dynamically identifies
the maintenance product using the `x_is_maintenance` ensuring the feature
works correctly for any properly configured maintenance product.

Task-5468757

closes odoo#1507

X-original-commit: d1777f8
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Dhrumil Shah (dhrs) <dhrs@odoo.com>
Before this commit the `product_barcodelookup` product attribute was unlinked,
which caused errors during the installation of the Hotel industry.

After this commit archives the attribute instead of deleting it, preventing
installation issues

Ticket-5472846

closes odoo#1510

X-original-commit: 18fa64a
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Dhrumil Shah (dhrs) <dhrs@odoo.com>
This commit adds an extended industry module for construction with:

- 'Show Line Numbering': On sale orders and invoices, modify views to
make the sequence numbers of the lines visible.
- 'Progress Management': In project, handle the SOL progress to easily
invoice customers (added in PR:1275).

Task-5135963

Part-of: odoo#1502
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
This commit moves the chatter down in order to display the full width
of the Sales Order Lines in the progress view.

task-5238284

closes odoo#1502

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
In this commit we have
- Rename attachment in the Driving School industry
- Add missing pictures in appointment of the Sports Club industry

Task-5449731

closes odoo#1522

X-original-commit: 7faaea6
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Dhrumil Shah (dhrs) <dhrs@odoo.com>
Before this commit, this test was relying on the state of the module
website_sale, but with the new runbot configuration (installing more
apps than what is in the dependencies), this test was wrong.

This commit changes the behaviour to rely on the dependency tree of the
module instead of the state.

closes odoo#1531

X-original-commit: c4f6467
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
We're adding new PO files for the following languages:
ar, az, bg, bs, ca, cs, da, de, el, es, es_419, et, fa, fi, fr, he, hi, hr, hu, id, it, ja, kab, ko, ku, lt, lv, mn, my, nb, nl, pl, pt, pt_BR, ro, ru, sk, sl, sr@latin, sv, th, tr, uk, vi, zh_CN, zh_TW
Add test to make sure a portal user has a correct and functional login (an email address)

closes odoo#1537

X-original-commit: 0307f77
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Add the house keeping feature to the booking_engine module, with its own setting so it can be (de)activated.
It is designed to allow the management of the cleaning and preparation of each room.
This provides a new board summarizing where house keeping stands for each room and a task list automatically populated based on ongoing stays.

TASK-5219678

closes odoo#1536

X-original-commit: 1e9aeb4
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: François Vasamillet (frva) <frva@odoo.com>
In this commit we update the industry module description so that it captures
the industry anatomy: Standard / Studio / Studio + Lines of code

Task-5410690

closes odoo#1540

X-original-commit: db2fffb
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
This commits make the city tax in booking engine optional, put behind a boolean on each product.
When it is not active, the popup will not appear to simplify the flow of the users who don't use it.

TASK-5419996

closes odoo#1549

X-original-commit: ba27c60
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: François Vasamillet (frva) <frva@odoo.com>
A recent heavy refactor in website_sale removed multiple product templates and
consolidated them into a single, class-based template.
Because of this change, several Industry websites were still referencing legacy
templates, resulting in a “Missing record: Template not found” warning on the
website.

This commit updates the affected Industry websites to align with the new
class-based design approach, ensuring all templates render correctly and the
warning is no longer displayed.

Task-5419879
COM PR - odoo/odoo#214744

closes odoo#1551

X-original-commit: 877ee72
Signed-off-by: Dhrumil Shah (dhrs) <dhrs@odoo.com>
vava-odoo and others added 18 commits March 2, 2026 18:30
Part-of: odoo#1776
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Part-of: odoo#1776
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
closes odoo#1776

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
The wellness practitioners industry relied on Studio only to send a simple
automated email.
This dependency was too costly for the value it brought to the customer.

Removed Studio from the module dependencies and deleted the related automation.

task - 5948518

closes odoo#1763

X-original-commit: 92b79d0
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Chirag Tribhovanbhai Gami (chga) <chga@odoo.com>
Validates:
- Sale order line quantity updated from maintained serials via automation
- Sale order serials aggregated from order lines
- FSM task inherits serials on creation
- FSM task serials updated on delivery validation
- Duplicate device serials blocked by server action
- New device creation triggers outgoing delivery
- Lot sale order count computed from related sale orders
- Lot project task count computed from related FSM tasks
- Lot delivery date computed from latest outgoing picking
- Lot maintenance sale orders aggregated from SO lines
- Lot maintenance state correctly resolved from deliveries and active
  subscriptions

Task- 5122516

closes odoo#1778

X-original-commit: f0a8cbb
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Chirag Tribhovanbhai Gami (chga) <chga@odoo.com>
Update run_industry.sh to reflect the refactor of IndustryUtils.__init__,
which no longer accepts industry_path as parameter.

closes odoo#1786

X-original-commit: 540383a
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
The issue occurred because x_unit_id is a non-stored related field
(x_product_id.uom_id). When a product was created on the fly, the related uom_id
value was not yet available, which resulted in a null value.

To resolve this, a new field `x_unit_custom_id` has been introduced. This field
is displayed conditionally when the product is being created dynamically and is
used instead of the related field to ensure a valid UoM is assigned.

The product creation logic has been updated to assign x_unit_custom_id instead
of x_unit_id, preventing the constraint error.

Fixed duplication issue in `server_action_create_or_modify_wi_on_sol`:
now writes back `x_sale_order_line_id` immediately after creating a new SOL.
This ensures subsequent edits update the existing SOL instead of creating
duplicates

Task-5097492

closes odoo#1787

X-original-commit: e3a2f14
Signed-off-by: Dhrumil Shah (dhrs) <dhrs@odoo.com>
closes odoo#1792

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
This commit adds an automation that automatically settles the POS orders (if any) of a customer when any invoice is created for them.

task-5929043

closes odoo#1715

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Signed-off-by: Tristan Cazier (trcaz) <trcaz@odoo.com>
If a slot is not linked to a sale.order.line, the automated action to
fix the slot time should not be triggered.

opw-6000739

closes odoo#1793

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
closes odoo#1802

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
The `trans_export` function was changed to require an environment
instead of a cursor as its last argument. This made it so the test
failed to export the POT files.

We update the function call here to use the correct argument.

closes odoo#1810

X-original-commit: 7510c15
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
Before this commit, the test to automatically export pot files was
looping over industry modules. But since saas-18.3, there are technical
submodules which are not included in the installed_industries variable.
This commit adapts the test to loop over installed_modules, which
includes all industry modules and technical modules.

closes odoo#1814

X-original-commit: 527dac6
Signed-off-by: Dylan Kiss (dyki) <dyki@odoo.com>
Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
In 19.1, standard odoo moved admin employee from data to demo data. Since it is needed it for demo data in industry modules, this commit creates a default admin employee demo data from base.user_admin for all DBs in base_industry_data and makes sure we don't conflict with any existing one.

task-5865586

closes odoo#1634

Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>
We're adding new PO files for the following languages:
ar, az, bg, bs, ca, cs, da, de, el, es, es_419, et, fa, fi, fr, he, hi, hr, hu, id, it, ja, kab, ko, ku, lt, lv, mn, my, nb, nl, pl, pt, pt_BR, ro, ru, sk, sl, sr@latin, sv, th, tr, uk, vi, zh_CN, zh_TW
* = ['booking_engine', 'booking_channex']

- Add channel manager setting in booking engine
- Install/uninstall booking_channex module via setting
- Add API key configuration
- Improve channex mapping model and views
- Update module dependencies

Task : 5913892
Copy link
Copy Markdown

@frva-odoo frva-odoo left a comment

Choose a reason for hiding this comment

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

Good refactoring, I only have a few comments. Great job !

<field name="name">Channex: Webhook action - manage booking create/change/delete</field>
<field name="model_id" ref="sale.model_sale_order" />
<field name="state">code</field>
<field name="code"><![CDATA[
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It is fine to remove it while you write the code to make it easier to read, but I think it's better to keep it afterward to avoid parsing errors in server actions

Comment on lines +64 to +66
sending_cron.message_post(
body="Here is answer from last channex restrictions update attempt: " + result
)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This was for debugging, I don't think we need it anymore now that we know it works. Either remove it or add a comment to remind us to remove it

Comment on lines +3 to +7
<menuitem id="mapping_menu_item"
name="Mappings"
parent="booking_engine.booking_engine_rooms_menu"
action="action_window_show_mapping_table"
/>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Don't forget to keep the group, this model shouldn't be seen by the average user of the db

- Change `x_model_type` values in channex server actions to match selection values.
- Minor refactoring.

task-5913987
@abdrahmanrashed abdrahmanrashed force-pushed the saas-19.1-booking-channex-mapping-aras branch from 0600e7d to fb8fb65 Compare April 14, 2026 06:55
Copy link
Copy Markdown

@frva-odoo frva-odoo left a comment

Choose a reason for hiding this comment

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

Perfect, good job !

@chga-odoo chga-odoo force-pushed the saas-19.1-booking-channex-setting-chga branch 7 times, most recently from 2051edf to 0dbfdb4 Compare April 15, 2026 10:33
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.