Skip to content

[MIG] report_substitute-V17 - #1

Merged
lef-adhoc merged 1 commit into
adhoc-dev:17.0-mig-report_substitutefrom
sodexis:17.0-mig-report_substitute
Apr 15, 2024
Merged

lef-adhoc merged 1 commit into
adhoc-dev:17.0-mig-report_substitutefrom
sodexis:17.0-mig-report_substitute

Conversation

@SodexisTeam

Copy link
Copy Markdown
  1. message_post_with_template is changed to message_post_with_source
  2. _onchange_template_id_wrapper is removed from base.

We don't need the method onchange_template_id_wrapper as whenever the template is changed in the wizard, _render_qweb_pdf is called through _generate_template_attachments while creating attachments which in turn substitutes the report correctly.

@lef-adhoc
lef-adhoc merged commit 29a842b into adhoc-dev:17.0-mig-report_substitute Apr 15, 2024
@lef-adhoc

lef-adhoc commented Aug 30, 2024

Copy link
Copy Markdown

@SodexisTeam Hello I have a problem in the method get_substitution_report_action in the line changed “action_report.read()[0]” to “self._for_xml_id(action_report.xml_id)”, what happens if it does not have xml_id ? For example if it was created by interface, I get error.

lef-adhoc pushed a commit that referenced this pull request Aug 30, 2024
* [IMP] is_materialized field non readonly on sql_valid state ; [FIX] block possibility to set indexes on non materialized view

* [FIX] set domain_force, group_ids readonly if state > sql_valid

* [IMP] better display of the field group_ids

* [IMP] possibility to reorder menu items from sql views

* [IMP] Do not warn user when setting sql view to draft if state is sql_valid

* [REF]

* [FIX] Set Date of the first execution in the action name
lef-adhoc pushed a commit that referenced this pull request Nov 28, 2025
* [IMP] is_materialized field non readonly on sql_valid state ; [FIX] block possibility to set indexes on non materialized view

* [FIX] set domain_force, group_ids readonly if state > sql_valid

* [IMP] better display of the field group_ids

* [IMP] possibility to reorder menu items from sql views

* [IMP] Do not warn user when setting sql view to draft if state is sql_valid

* [REF]

* [FIX] Set Date of the first execution in the action name
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.

3 participants