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

[IMP] inventory: fix visibility days #11444

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open

[IMP] inventory: fix visibility days #11444

wants to merge 1 commit into from

Conversation

Felicious
Copy link
Contributor

@Felicious Felicious commented Nov 22, 2024 •
edited
Loading

Fix inaccuracies about visibility days explanation, with improved example and supportive imagery to visualize how the dates work together.

Changelog

  • restructure reordering rules documentation by pulling the section about how the replenishment report works into a new document
  • Explain just-in-time logic
  • Explain how the Forecasted quantities are determined
  • Create custom visualizations for visibility days and forecasted date
  • Backport explanation about days to purchase (may run into merge conflicts when fwporting because of the internal links' naming scheme!)

FWport: to master

@Felicious Felicious added the 2 label Nov 22, 2024
@Felicious Felicious self-assigned this Nov 22, 2024
@robodoo
Copy link
Collaborator

robodoo commented Nov 22, 2024

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team November 22, 2024 01:40
@Felicious Felicious marked this pull request as draft November 22, 2024 14:41
@Felicious
Copy link
Contributor Author

The BE expert reached out and told me there's still stuff for me to improve 🙈 I'll ping you on discord when this PR is ready, @hojo-odoo !

@Felicious Felicious marked this pull request as ready for review November 27, 2024 03:33
@Felicious Felicious force-pushed the 16.0-inventory-fix-visibility-days-feku branch from 3ac0dbd to e75a8dd Compare November 27, 2024 03:45
@Felicious
Copy link
Contributor Author

Hi @hojo-odoo ! This PR is ready for review!

https://runbot158.odoo.com/runbot/static/build/71189420-16-0/logs/build/html/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/report.html

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

@crl-odoo crl-odoo Awaiting requested review from crl-odoo

@hojo-odoo hojo-odoo Awaiting requested review from hojo-odoo

At least 1 approving review is required to merge this pull request.

Assignees

@Felicious Felicious

Labels
5 16.0 17.0 18.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Felicious @robodoo