[ADD] account_credit_control: migration scripts #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
It was working, but better to not introduce such typo.
…html field change
(cherry picked from commit 811310fd06ca8589fa8e87da6b8bdaea403bf660) [MIG] Improve the code and fix some bugs (cherry picked from commit 5bcdfe3f748884d650120f76d1642b820d0a800b) [FIX] Change the parent for the menu Credit Control (the previous parent menu was not in the Odoo standard) (cherry picked from commit 826257ce7b5248956d838cae9c94a012d550327e) [FIX] Fix a lot of errors to respect pep8 (cherry picked from commit bed6cf35d665eea32122c91d33322af141053f29) [FIX] Respect PEP8 (cherry picked from commit 6353eef0dce1bfe5db8a38d3410d1cc7f833fa10)
…useless spaces, ...) and fix some small bugs (cherry picked from commit efd9e9f8efe9a5e8da791d179b5a5d57710a31c3) [FIX] Add space to respect PEP8 (cherry picked from commit 06c93c610646bb4269f303c3a7a55700cc6dfd1f) [FIX] Replace the field "full_reconcile_id" by "reconciled" in a search domain on account.move.line (cherry picked from commit cf2cfd43e34dff9eef2571b87de777a183c220c0)
Let's use OU populated old_invoice_id field for taking into account draft invoices converted to account.move records.
If not, the FK constraint is kept and thus we can't switch the values.
Before this patch, `credit.control.communication` was a `TransientModel` that sent emails. This produced that, when someone responded to one of those emails, Odoo detected it was a response but couldn't know where to store it, so it was lost. Also, users with credit control access got a 500 error when clicking on the "View" button in the email. Now, `credit.control.communication` is a normal `Model` which inherits from `mail.thread` and `mail.activity.mixin`. This makes more sense, because when you are tracking credit control communications, you usually communicate with a customer regarding several lines at once (no sense on asking a customer to pay each line separately). More changes: - `credit.control.run` now displays an additional smart button that leads users to generated communications. - `credit.control.line` now is linked to a communication instead of a simple message (the old link is preserved in a hidden legacy column). - `credit.control.line` has a new state "queued", which indicates that the communication is executed but still not sent. When sent, it's changed to "done" or "email_error". - Comunication emails have a specific subtype. - Some fields are renamed, as they didn't follow guidelines. - Some menu names are less verbose now. - Mail templates code is migrated. @Tecnativa TT24841
- Template translations migration scripts - Template recipients
Update patch imported from OCA#86 to new style guidelines.
Adapt imported patch OCA#86 to v13. @Tecnativa TT24841
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): | ||
| # Skip migration if 12.0.3 was already installed (migration was already run) | ||
| if parse_version("12.0.3.0.0") <= parse_version(version) < parse_version("13.0"): | ||
| return |
There was a problem hiding this comment.
Ho visto che hai saltato anche gli script precedenti: https://github.com/OCA/credit-control/tree/12.0/account_credit_control/migrations/12.0.1.7.0.
E' voluto?
There was a problem hiding this comment.
nel task ci sono dei commenti a riguardo
There was a problem hiding this comment.
Ok grazie, non lo avevo notato perché barrato.
SimoneVagile
approved these changes
Feb 25, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.