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.

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

[16.0][ADD] stock_storage_category_usage_report #352

Open
wants to merge 5 commits into
base: 16.0
Choose a base branch
from

Conversation

@rousseldenis rousseldenis added this to the 16.0 milestone Nov 13, 2024
@rousseldenis rousseldenis force-pushed the 16.0-add-stock-storage-category-report-dro branch from 400afc6 to e05d1f3 Compare November 13, 2024 10:43
@rousseldenis rousseldenis force-pushed the 16.0-add-stock-storage-category-report-dro branch from e05d1f3 to dd30f32 Compare November 13, 2024 10:49
Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

Side note and I'm not an expert in English but I think occupy is more about persons, like a room being occupied. I would have used filled_location_count and fill_rate

@rousseldenis rousseldenis force-pushed the 16.0-add-stock-storage-category-report-dro branch 2 times, most recently from 33dd8cb to 243db31 Compare November 15, 2024 09:44
@rousseldenis rousseldenis force-pushed the 16.0-add-stock-storage-category-report-dro branch 4 times, most recently from b68aaa0 to 6f6733f Compare November 19, 2024 13:54
@rousseldenis rousseldenis force-pushed the 16.0-add-stock-storage-category-report-dro branch from 6f6733f to d454ebc Compare November 19, 2024 13:55
Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

And I'm happy to be maintainer, co-author... :)

# (move qty is done but not yet validated).
# We don't take into account filled and being filled moves.
empty_locations = all_locations.filtered(
lambda location: location.fill_state == "empty"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
lambda location: location.fill_state == "empty"
lambda location: location.fill_state in ("empty", "being_emptied")

Comment on lines +5 to +8
"name": "Stock Storage Category Usage Report",
"summary": """
This module allows to get reporting on stock storage categories
about their locations usage""",
Copy link
Contributor

Choose a reason for hiding this comment

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

What about renaming to use fill instead of usage as this relates to that field?

Suggested change
"name": "Stock Storage Category Usage Report",
"summary": """
This module allows to get reporting on stock storage categories
about their locations usage""",
"name": "Stock Storage Category Fill Report",
"summary": """
This module allows to get reporting on stock storage categories
about their locations fill rate""",

@@ -0,0 +1,2 @@
When users want to manage their stock locations in a better way,
they need those locations' usage (empty/filled in).
Copy link
Contributor

@jbaudoux jbaudoux Nov 20, 2024

Choose a reason for hiding this comment

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

Maybe DESCRIPTION is enough. It's kind of repeating

Suggested change
they need those locations' usage (empty/filled in).
they need those location storage categories' fill rate (amount of empty/filled location).

#: model:ir.ui.menu,name:stock_storage_category_usage_report.report_storage_category_root_menu
#: model:ir.ui.menu,name:stock_storage_category_usage_report.report_storage_category_view_menu
msgid "Stock Storage Category Usage Report"
msgstr "Rapport d'utilisation des catégories d'empalcement"
Copy link
Contributor

@jbaudoux jbaudoux Nov 20, 2024

Choose a reason for hiding this comment

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

:) ou Rapport de taux de remplissage

Suggested change
msgstr "Rapport d'utilisation des catégories d'empalcement"
msgstr "Rapport d'utilisation des catégories d'emplacements"

#. module: stock_storage_category_usage_report
#: model:ir.model,name:stock_storage_category_usage_report.model_stock_storage_category
msgid "Storage Category"
msgstr "Catégories d'emplacement"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
msgstr "Catégories d'emplacement"
msgstr "Catégorie d'emplacement"

@@ -0,0 +1,73 @@
# Translation of Odoo Server.
Copy link
Contributor

Choose a reason for hiding this comment

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

Rename file to fr.po

</record>

<record model="ir.actions.act_window" id="report_storage_category_usage_act_window">
<field name="name">Stock Storage Category Usage Report</field>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<field name="name">Stock Storage Category Usage Report</field>
<field name="name">Storage Categories Fill Rate Report</field>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants