Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[15.0][MIG] sale_order_revision #2381

Merged
merged 30 commits into from
May 8, 2023
Merged

[15.0][MIG] sale_order_revision #2381

merged 30 commits into from
May 8, 2023

Conversation

CRogos
Copy link
Contributor

@CRogos CRogos commented Feb 17, 2023

Followup of #2275

  • Renamed test_requirements
  • removed oca_requirements
  • squashed commits

@simahawk @grindtildeath @cyrilmanuel @leemannd

rvalyi and others added 30 commits November 23, 2022 10:03
@rvalyi @leemannd
[ADD] sale_order_revision, modeled after purchase_order_revision OCA …
b2b35bf 
…module by Agile Business Group
@gurneyalex @leemannd
[FIX+IMP] sale_order_revision: Several things:
0b34375 
* extract description from __openerp__.py
* move things according to standard OCA layout
* reformatting
* port to new API
* Major rewrite of the 7.0 version:

  * the name of the SO is kept, and a revision number is appended
  * old revisions are made inactive
  * rewrite the copy logic

* update translations, added en.po fr.po
* fix typo in README
* use post_init_hook
* more assertions in the test
* sale_order_revision: remove en.po
* current_revision is only visible for non current revisions
@leemannd
Added german po file
104ac08
@sysadminmatmoz @leemannd
Slovene translations added
db1112b
@oca-transbot @leemannd
OCA Transbot updated translations from Transifex
e3f8ad2
@JayVora-SerpentCS @leemannd
[MIG] sale_order_revision: Migration to v10
6793688
@grindtildeath @leemannd
[FIX] sale_order_revision: 2 things:
4cfc3a1 
* Fix create revision and improve view invisibility
* Fix unrevisioned name and improve test
@gurneyalex @leemannd
[FIX] sale_order_revision: wrong keyword (OCA#554)
fd02145 
wrong keyword name in SaleOrder.copy definition
was breaking sale order duplication
@oca-travis @leemannd
[UPD] Update sale_order_revision.pot
103e4f2
@devang-dreambits @leemannd
[MIG] sale_order_revision: Migration to 11.0
f7e91cb
@simonitfi @leemannd
Hide Revisions-tab entirely if there is not any revisions
3145b4c
@rven @leemannd
[MIG] sale_order_revision: Migration to 12.0
eed191e
@oca-travis @leemannd
[UPD] Update sale_order_revision.pot
5147e3f
@oca-transbot @leemannd
Update translation files
a0bacf8 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/
@marylla @leemannd
Translated using Weblate (German)
22a8cfd 
Currently translated at 100.0% (14 of 14 strings)

Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/de/
@OCA-git-bot @leemannd
[UPD] README.rst
3f28aca
@liweijie0812 @leemannd
Translated using Weblate (Chinese (Simplified))
360cce5 
Currently translated at 100.0% (14 of 14 strings)

Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/zh_CN/
@Roodin @leemannd
Translated using Weblate (Spanish)
964ffc4 
Currently translated at 100.0% (14 of 14 strings)

Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/es/
@leemannd
[MIG] sale_order_revision: Migration to 13.0
71dbfe5 
[13.0] fix pre-commit + formatting

add test lines and linting xml
@oca-travis @leemannd
[UPD] Update sale_order_revision.pot
11e2811 
[UPD] README.rst
@oca-transbot @leemannd
Update translation files
d489760 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-13.0/sale-workflow-13.0-sale_order_revision
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-13-0/sale-workflow-13-0-sale_order_revision/
@kittiu @leemannd
[14.0][MIG] sale_order_revision
afc8a86
@oca-travis @leemannd
[UPD] Update sale_order_revision.pot
5740298 
[UPD] README.rst
@victoralmau @leemannd
[IMP] sale_order_revision: Tests are added to check that the addon wo…
d859046 
…rks as expected.

TT33433
@OCA-git-bot @leemannd
sale_order_revision 14.0.1.0.1
815bd40
@victoralmau @leemannd
[IMP] sale_order_revision: Set state=cancel in _prepare_revision_data…
b97483e 
…() function.
@oca-travis @leemannd
[UPD] Update sale_order_revision.pot
cee93d1
@OCA-git-bot @leemannd
sale_order_revision 14.0.1.1.0
36dea40
@jwaes @leemannd
[IMP] sale_order_revision: black, isort, prettier
f7302eb
[MIG] sale_order_revision: Migration to 15.0
2651da3
@OCA-git-bot
Copy link
Contributor

Hi @OriolVForgeFlow,
some modules you are maintaining are being modified, check this out!

@CRogos CRogos marked this pull request as ready for review February 17, 2023 15:00
@CRogos CRogos mentioned this pull request Feb 17, 2023
Merged
@rousseldenis
Copy link
Sponsor Contributor

/ocabot migration sale_order_revision

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Apr 19, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 19, 2023
Open
@OCA-git-bot
Copy link
Contributor

The migration issue (#1741) has been updated to reference the current pull request.
however, a previous pull request was referenced : #1931.
Perhaps you should check that there is no duplicate work.
CC : @jwaes

rousseldenis
rousseldenis approved these changes Apr 19, 2023
View reviewed changes
Copy link
Sponsor Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Code review

@simahawk simahawk mentioned this pull request Apr 19, 2023
Closed
@CRogos
Copy link
Contributor Author

CRogos commented May 8, 2023

@pedrobaeza can you merge this?

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-2381-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 8, 2023
@OCA-git-bot
Merge PR #2381 into 15.0
64e42f5 
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 15.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 15.0-ocabot-merge-pr-2381-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 005ee2a into OCA:15.0 May 8, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 7cd3a76. Thanks a lot for contributing to OCA. ❤️

@CRogos CRogos deleted the 15.0-mig-sale_order_revision branch May 8, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leemannd leemannd leemannd approved these changes

@rousseldenis rousseldenis rousseldenis approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 migration needs review
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.

None yet