Skip to content

Conversation

@github-actions
Copy link

@github-actions github-actions bot commented Jul 8, 2025

Analysis or noupdate changes for modules marked as done:

  • account
  • account_edi_ubl_cii
  • base
  • event
  • hr
  • mail
  • product
  • web_hierarchy

@github-actions github-actions bot added this to the 18.0 milestone Jul 8, 2025
@pedrobaeza
Copy link
Member

All the changes are elements reordering. Is this going to happen continuously?

@github-actions github-actions bot force-pushed the 18.0-update-analysis-bot branch from 4ffea93 to 431ea0b Compare July 14, 2025 05:30
@pedrobaeza pedrobaeza merged commit e0a410c into 18.0 Jul 14, 2025
@OCA-git-bot OCA-git-bot deleted the 18.0-update-analysis-bot branch July 14, 2025 07:49
account / account.move / audit_trail_message_ids (one2many): NEW relation: mail.message
account / account.move / checked (boolean) : NEW
account / account.move / invoice_currency_rate (float) : NEW isfunction: function, stored
account / account.move / invoice_currency_rate (float) : NEW hasdefault: compute
Copy link
Member

Choose a reason for hiding this comment

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

Changed in odoo/odoo@2d486fb. Anyway, previously, if not renamed, it was needed to be pre-added and computed via SQL. Can you check, @MiquelRForgeFlow ?

Copy link
Contributor

Choose a reason for hiding this comment

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

In that commit, invoice_currency_rate was made editable, but is hasn't changed in the way it is computed.

But, regarding to pre-creating it, I think I avoided to do it due the difficulty to convert the compute into SQL.

Feel free to improve the code :)

Copy link
Member

@pedrobaeza pedrobaeza Jul 14, 2025

Choose a reason for hiding this comment

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

@MiquelRForgeFlow that's something you can't avoid, because if not, migrating from 17 to 18 with thousand of moves will trigger the computation and takes hours. Please complete the account migration with such thing, as it can be considered essential.

Copy link
Contributor

Choose a reason for hiding this comment

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

Well, I don't have a clear way to do it right now. The needed query isn't simple.

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