Skip to content

Fix construction cart habitation warning false positive#413

Merged
jplacht merged 3 commits intoPRUNplanner:mainfrom
richjenks:fix-construction-cart-hab-warn
Apr 28, 2026
Merged

Fix construction cart habitation warning false positive#413
jplacht merged 3 commits intoPRUNplanner:mainfrom
richjenks:fix-construction-cart-hab-warn

Conversation

@richjenks
Copy link
Copy Markdown
Contributor

@richjenks richjenks commented Apr 23, 2026

#390 can false positive because it only looks at Amount, not Amount + Built. Example:

image

This PR fixes the calculations and adds a tooltip so users know what's gone wrong:

image

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2026

Deploy Preview for prunplanner-preview ready!

Name Link
🔨 Latest commit 59573e0
🔍 Latest deploy log https://app.netlify.com/projects/prunplanner-preview/deploys/69ea0f05d135f4000777c6d1
😎 Deploy Preview https://deploy-preview-413--prunplanner-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes. Give us feedback

@jplacht jplacht added the improve Improves existing functionalities label Apr 27, 2026
@jplacht jplacht merged commit 16f82b8 into PRUNplanner:main Apr 28, 2026
9 of 10 checks passed
@richjenks richjenks deleted the fix-construction-cart-hab-warn branch April 28, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improve Improves existing functionalities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants