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

Approval #5545

Closed
wants to merge 15 commits into from
Closed

Approval #5545

wants to merge 15 commits into from

Conversation

matmair
Copy link
Member

@matmair matmair commented Sep 15, 2023

Follow up to #5095

Simplified implementation of approvals for orders. Includes building blocks for a more generalized rules and templates driven approach.

Function

This PR adds approvals (optionally) for POs. You can enable them in settings (including minimum approvers and maximum rejections). After that POs can only be placed once they are approved.

Notifications

Notifications will be sent out to the responsible user/group:

  • when approval starts
  • on each approval decision
  • after being rejected (PO will be cancelled)
  • after being approved (PO will be moved to placed)

UI

As an experiment, I just implemented this in P UI. While the order views are not ready in P UI, approvals are.

Next steps

This is an MVP, I opened an epic for further improvements #5097.
While this PR adds a business logic and UI for PO approvals, approvals are a first-class citizen implementation-wise. They can be used as-is for any model and adding UI elements for that should be simple. The default rules engine is also simple right now but designed for complex requirements. If there is interest this can be extended.

Still to address

Preview Give feedback

Closes #3501

@netlify
Copy link

netlify bot commented Sep 15, 2023

Deploy Preview for inventree ready!

Name Link
🔨 Latest commit 761c133
🔍 Latest deploy log https://app.netlify.com/sites/inventree/deploys/6503a58410759b0008a474ec
😎 Deploy Preview https://deploy-preview-5545--inventree.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100
Accessibility: 86
Best Practices: 100
SEO: 70
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@matmair matmair self-assigned this Sep 15, 2023
@matmair matmair added enhancement This is an suggested enhancement or new feature order Related to purchase orders / sales orders api Relates to the API labels Sep 15, 2023
@matmair matmair added this to the 0.13.0 milestone Sep 15, 2023
@matmair matmair closed this Sep 17, 2023
@matmair matmair deleted the approval-upstream branch May 27, 2024 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Relates to the API enhancement This is an suggested enhancement or new feature order Related to purchase orders / sales orders
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FR] Purchase Order Approvals
1 participant