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] stock_quantity_history_location: Migration to version 15.0 #272

Merged
merged 19 commits into from
Mar 29, 2023
Merged

[15.0][MIG] stock_quantity_history_location: Migration to version 15.0 #272

merged 19 commits into from
Mar 29, 2023

Conversation

ernestotejeda
Copy link
Member

@ernestotejeda ernestotejeda commented Mar 29, 2023 •
edited
Loading

Supersede #202

  • Inherit the template to change the button name
  • Modify action["display_name"] instead of action["name"]
  • Add attrs="{'invisible': [('location_id', '=', False)]}" to the 'Include child locations' field in its XML view definition
  • Squash administrative commits to reduce the commit history

Cc @Tecnativa TT37324

ping @pedrobaeza @chienandalu @desdelinux @rousseldenis

JordiBForgeFlow and others added 18 commits March 29, 2023 13:30
@JordiBForgeFlow
[11.0][ADD] stock_quantity_history_location & stock_account_quantity_…
367548d 
…history_location
@HviorForgeFlow
[MIG] stock_quantity_history_location: Migration to 12.0
d843146
@HviorForgeFlow
[FIX] stock_quantity_history_location: stock.quant action fixup!
a269450
@HviorForgeFlow
[IMP] stock_quantity_history_location: black, isort
b00a047
@HviorForgeFlow
[MIG] stock_quantity_history_location: Migration to 13.0
bd35ca6
@HviorForgeFlow
[13.0][IMP] stock_quantity_history_location: Block location selection…
e82c429 
… on Account Valuation
@erik-bzcl
Added translation using Weblate (Romanian)
cf44efe
@erik-bzcl
Translated using Weblate (Romanian)
e998c10 
Currently translated at 100.0% (4 of 4 strings)

Translation: stock-logistics-reporting-13.0/stock-logistics-reporting-13.0-stock_quantity_history_location
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-13-0/stock-logistics-reporting-13-0-stock_quantity_history_location/ro/
@joao-p-marques
Apply pre-commit
2df0f1e
@alessandro-fiorino
Added translation using Weblate (Italian)
fb57ee8
@alessandro-fiorino
Translated using Weblate (Italian)
91f8767 
Currently translated at 100.0% (4 of 4 strings)

Translation: stock-logistics-reporting-13.0/stock-logistics-reporting-13.0-stock_quantity_history_location
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-13-0/stock-logistics-reporting-13-0-stock_quantity_history_location/it/
@victoralmau
[IMP] stock_quantity_history_location: Add functions in test to allow…
d2b378d 
… re-use by othher addons
@victoralmau
[FIX] stock_quantity_history_location: Add literal_eval to prevent er…
9f472b0 
…rors according to stock_account_quantity_history_location
@atchuthan
[IMP] stock_quantity_history_location: black, isort, prettier
354f6b1
@atchuthan
[MIG] stock_quantity_history_location: Migration to 14.0
dbd68ea
@kakurai8
Added translation using Weblate (Japanese)
a0e2d07
@kakurai8
Translated using Weblate (Japanese)
5d9cc60 
Currently translated at 100.0% (6 of 6 strings)

Translation: stock-logistics-reporting-14.0/stock-logistics-reporting-14.0-stock_quantity_history_location
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-14-0/stock-logistics-reporting-14-0-stock_quantity_history_location/ja/
@AnnaPForgeFlow
[IMP] stock_quantity_history_location: black, isort, prettier
a32ca85
sergio-teruel
sergio-teruel reviewed Mar 29, 2023
View reviewed changes
stock_quantity_history_location/static/src/xml/inventory_report.xml
<?xml version="1.0" encoding="UTF-8" ?>
<templates>
<t t-extend="StockInventoryReport.Buttons">
<t t-jquery="button.o_button_at_date" t-operation="inner">
Copy link
Contributor

Choose a reason for hiding this comment

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

We only must change the button description if we have group_stock_multi_locations security group.

Copy link
Member Author

Choose a reason for hiding this comment

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

Right!

sergio-teruel
sergio-teruel approved these changes Mar 29, 2023
View reviewed changes
Copy link
Contributor

@sergio-teruel sergio-teruel left a comment

Choose a reason for hiding this comment

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

Only little comment

@pedrobaeza pedrobaeza added this to the 15.0 milestone Mar 29, 2023
@pedrobaeza pedrobaeza mentioned this pull request Mar 29, 2023
Closed
@pedrobaeza
Copy link
Member

/ocabot migration stock_quantity_history_location

@OCA-git-bot OCA-git-bot mentioned this pull request Mar 29, 2023
Closed
13 tasks
@OCA-git-bot
Copy link
Contributor

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

This was referenced Mar 29, 2023
Closed
Closed
@AnnaPForgeFlow @DavidJForgeFlow
[MIG] stock_quantity_history_location: Migration 15.0
45e9880 
Co-authored-by: DavidJForgeFlow <david.jimenez@forgeflow.com>
@ernestotejeda ernestotejeda force-pushed the 15.0-mig-stock_quantity_history_location branch from e382125 to 45e9880 Compare March 29, 2023 21:23
@ernestotejeda
Copy link
Member Author

changes done @sergio-teruel

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-272-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b580b22 into OCA:15.0 Mar 29, 2023
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 15.0-mig-stock_quantity_history_location branch March 29, 2023 22:44
@pedrobaeza pedrobaeza mentioned this pull request Mar 29, 2023
Merged
1 task
@desdelinux
Copy link

Hello @ernestotejeda , are you going to migrate this to 16.0?

@chienandalu
Copy link
Member

Not soon @desdelinux . Feel free to do it 😄 (it shouldn't be much effort I think)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chienandalu chienandalu chienandalu approved these changes

@sergio-teruel sergio-teruel sergio-teruel approved these changes

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

Successfully merging this pull request may close these issues.

None yet