[17.0][MIG] stock_quantity_history_location #316
Merged
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
AnaisLForgeFlow
force-pushed
the
17.0-mig-stock_quantity_history_location
branch
from
61344a9
to
1efe2c6
Compare
GuillemCForgeFlow
suggested changes
May 14, 2024
stock_quantity_history_location/wizards/stock_quantity_history.xml
Outdated
Show resolved
Hide resolved
AnaisLForgeFlow
force-pushed
the
17.0-mig-stock_quantity_history_location
branch
from
1efe2c6
to
760f848
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
|
/ocabot migration stock_quantity_history_location |
|
|
||
| @api.model | ||
| def get_views(self, views, options=None): | ||
| res = super().get_views(views, options=options) |
There was a problem hiding this comment.
@AnaisLForgeFlow Maybe a good improvement should be adding a little test calling the view.
There was a problem hiding this comment.
There was a problem hiding this comment.
@rousseldenis sure, we can add this for better test coverage
There was a problem hiding this comment.
@rousseldenis I've just added the test case, I believe it should be ready to merge 🙂
rousseldenis
approved these changes
Sep 20, 2024
github-actions
bot
removed
the
stale
PieterPaulussen
approved these changes
Sep 25, 2024
|
@GuillemCForgeFlow can you update your review? |
atchuthan
approved these changes
Oct 21, 2024
… on Account Valuation
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/
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/
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/
Co-authored-by: DavidJForgeFlow <david.jimenez@forgeflow.com>
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-reporting-15.0/stock-logistics-reporting-15.0-stock_quantity_history_location Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-15-0/stock-logistics-reporting-15-0-stock_quantity_history_location/
Currently translated at 60.0% (3 of 5 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_quantity_history_location Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_quantity_history_location/it/
This commit solves the error "Non loaded module stock_quantity_history_location.InventoryReportListController" because the dependency stock.InventoryReportListController was missed. The dependency from stock was migrated to OWL [1], so it is inherited using the OWL to preserve the behavior. References: - [1] odoo/odoo@ec0b1005
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_quantity_history_location Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_quantity_history_location/
Currently translated at 100.0% (5 of 5 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_quantity_history_location Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_quantity_history_location/it/
Currently translated at 100.0% (5 of 5 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_quantity_history_location Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_quantity_history_location/es/
GuillemCForgeFlow
force-pushed
the
17.0-mig-stock_quantity_history_location
branch
from
760f848
to
c14f7c1
Compare
- refactored .js code by inheriting `get_views` method to display a different button text for those users having the Multi Locations group enabled. Added a test case to cover the change
GuillemCForgeFlow
force-pushed
the
17.0-mig-stock_quantity_history_location
branch
from
c14f7c1
to
f28ee0e
Compare
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 2536cc1. Thanks a lot for contributing to OCA. ❤️ |
GuillemCForgeFlow
deleted the
17.0-mig-stock_quantity_history_location
branch
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.
Standard Migration
@ForgeFlow